EVEREST-1063 Schedules validation (#337)

consider only the schedule regardless of the storage
percona · May 28, 2024 · 1432ab5 · 1432ab5
1 parent 497a382
commit 1432ab5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/api/validation.go b/api/validation.go
@@ -762,7 +762,7 @@ type apiSchedule []struct {
 func checkDuplicateSchedules(schedules apiSchedule) error {
 	m := make(map[string]struct{})
 	for _, s := range schedules {
-		key := s.Schedule + s.BackupStorageName
+		key := s.Schedule
 		if _, ok := m[key]; ok {
 			return errDuplicatedSchedules
 		}

diff --git a/api/validation_test.go b/api/validation_test.go
@@ -422,7 +422,7 @@ func TestValidateBackupSpec(t *testing.T) {
 		},
 		{
 			name:    "errDuplicatedSchedules",
-			cluster: []byte(`{"spec": {"backup": {"enabled": true, "schedules": [{"schedule": "0 0 * * *", "name": "name"}, {"schedule": "0 0 * * *", "name": "name"}]}}}`),
+			cluster: []byte(`{"spec": {"backup": {"enabled": true, "schedules": [{"schedule": "0 0 * * *", "name": "name"}, {"schedule": "0 0 * * *", "name": "otherName"}]}}}`),
 			err:     errDuplicatedSchedules,
 		},
 		{