delete old backup objects only if successfully completed

digitalocean · Apr 19, 2022 · 5929559 · 5929559
1 parent cd13dd3
commit 5929559
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/controllers/etcdbackupschedule_controller.go b/controllers/etcdbackupschedule_controller.go
@@ -196,7 +196,7 @@ func (r *EtcdBackupScheduleReconciler) cleanup(resourceNamespacedName types.Name
 	for i, backup := range etcdBackups.Items {
 		backup := backup
 		funcs[i] = func() error {
-			if cleanAll || backup.Status.Phase != etcdv1alpha1.EtcdBackupPhaseBackingUp {
+			if cleanAll || backup.Status.Phase == etcdv1alpha1.EtcdBackupPhaseCompleted {
 				if err := r.Delete(ctx, &backup); err != nil {
 					return client.IgnoreNotFound(err)
 				}