From f05c6a8beb8fdf53d18992d37a837ec4124fa48a Mon Sep 17 00:00:00 2001 From: Kevin Date: Fri, 25 Aug 2023 15:56:34 -0400 Subject: [PATCH] renamed variable to avoid double negation Signed-off-by: Kevin --- pkg/controller/queuejob/queuejob_controller_ex.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/controller/queuejob/queuejob_controller_ex.go b/pkg/controller/queuejob/queuejob_controller_ex.go index cdf8ab53..011d4504 100644 --- 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)