Fix incorrect usage of klog .*f functions
The .*f variants should only ever be called with arguments to format.
This commit is contained in:
parent
4d294562e5
commit
780e68f6d2
|
@ -111,13 +111,13 @@ func (e *BinpackingNodeEstimator) Estimate(
|
||||||
|
|
||||||
remainingPods, err = e.tryToScheduleOnExistingNodes(estimationState, podsEquivalenceGroup.Pods)
|
remainingPods, err = e.tryToScheduleOnExistingNodes(estimationState, podsEquivalenceGroup.Pods)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
klog.Errorf(err.Error())
|
klog.Error(err.Error())
|
||||||
return 0, nil
|
return 0, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
err = e.tryToScheduleOnNewNodes(estimationState, nodeTemplate, remainingPods)
|
err = e.tryToScheduleOnNewNodes(estimationState, nodeTemplate, remainingPods)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
klog.Errorf(err.Error())
|
klog.Error(err.Error())
|
||||||
return 0, nil
|
return 0, nil
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -139,7 +139,7 @@ func (t *LoopTrigger) logTriggerReason(message string) {
|
||||||
case <-t.podObserver.unschedulablePodChan:
|
case <-t.podObserver.unschedulablePodChan:
|
||||||
klog.Info("Autoscaler loop triggered by unschedulable pod appearing")
|
klog.Info("Autoscaler loop triggered by unschedulable pod appearing")
|
||||||
default:
|
default:
|
||||||
klog.Infof(message)
|
klog.Info(message)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue