fix: delete active jobs right away when job finishes even when TTLSec…

…ondsAfterFinished is set
kubernetes-sigs · Aug 27, 2024 · c3581cb · c3581cb
1 parent 2dcc751
commit c3581cb
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/pkg/controllers/jobset_controller.go b/pkg/controllers/jobset_controller.go
@@ -159,13 +159,13 @@ func (r *JobSetReconciler) reconcile(ctx context.Context, js *jobset.JobSet, upd
 			log.Error(err, "executing ttl after finished policy")
 			return ctrl.Result{}, err
 		}
-		if requeueAfter > 0 {
-			return ctrl.Result{RequeueAfter: requeueAfter}, nil
-		}
 		if err := r.deleteJobs(ctx, ownedJobs.active); err != nil {
 			log.Error(err, "deleting jobs")
 			return ctrl.Result{}, err
 		}
+		if requeueAfter > 0 {
+			return ctrl.Result{RequeueAfter: requeueAfter}, nil
+		}
 		return ctrl.Result{}, nil
 	}