Merge pull request #10 from adrianchiris/minor-sched-log-change

minor logging change in scheduler controller
Mellanox · Aug 21, 2024 · 72b91bb · 72b91bb
2 parents 7ccc345 + bb730ee
commit 72b91bb
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/internal/controller/nodemaintenancescheduler_controller.go b/internal/controller/nodemaintenancescheduler_controller.go
@@ -158,8 +158,8 @@ func (r *NodeMaintenanceSchedulerReconciler) Reconcile(ctx context.Context, req
 		return ctrl.Result{}, err
 	}
 
-	if schedCtx.AvailableSlots == 0 {
-		r.Log.Info("no slots available for maintenance scheduling")
+	if len(schedCtx.CandidateMaintenance) == 0 {
+		r.Log.Info("no candidate NodeMaintenance requests")
 		return schedulerResyncResult, nil
 	}
 
@@ -168,8 +168,8 @@ func (r *NodeMaintenanceSchedulerReconciler) Reconcile(ctx context.Context, req
 		return schedulerResyncResult, nil
 	}
 
-	if len(schedCtx.CandidateMaintenance) == 0 {
-		r.Log.Info("no candidate NodeMaintenance requests")
+	if schedCtx.AvailableSlots == 0 {
+		r.Log.Info("no slots available for maintenance scheduling")
 		return schedulerResyncResult, nil
 	}