renamed variable to avoid double negation

Signed-off-by: Kevin <[email protected]>
project-codeflare · Aug 26, 2023 · f05c6a8 · f05c6a8
1 parent 59bab13
commit f05c6a8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/controller/queuejob/queuejob_controller_ex.go b/pkg/controller/queuejob/queuejob_controller_ex.go
@@ -521,9 +521,9 @@ func (qjm *XController) GetQueueJobsEligibleForPreemption() []*arbv1.AppWrapper
 				awDispatchDurationLimit := value.Spec.SchedSpec.DispatchDuration.Limit
 				dispatchDuration := value.Status.ControllerFirstDispatchTimestamp.Add(time.Duration(awDispatchDurationLimit) * time.Second)
 				currentTime := time.Now()
-				hasDispatchTimeNotExceeded := currentTime.Before(dispatchDuration)
+				dispatchTimeExceeded := !currentTime.Before(dispatchDuration)
 
-				if !hasDispatchTimeNotExceeded {
+				if dispatchTimeExceeded {
 					klog.V(8).Infof("Appwrapper Dispatch limit exceeded, currentTime %v, dispatchTimeInSeconds %v", currentTime, dispatchDuration)
 					value.Spec.SchedSpec.DispatchDuration.Overrun = true
 					qjobs = append(qjobs, value)