Skip to content

Commit 82d27aa

Browse files
authored
Merge pull request kubernetes#91832 from ahg-g/ahg-over
Remove log message causing significant overhead on Preemption evaluation
2 parents 3a8d130 + 8bf3053 commit 82d27aa

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

pkg/scheduler/core/generic_scheduler.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -273,6 +273,13 @@ func (g *genericScheduler) Preempt(ctx context.Context, prof *profile.Profile, s
273273
// In this case, we should clean-up any existing nominated node name of the pod.
274274
return "", nil, []*v1.Pod{pod}, nil
275275
}
276+
if klog.V(5).Enabled() {
277+
var sample []string
278+
for i := 0; i < 10 && i < len(potentialNodes); i++ {
279+
sample = append(sample, potentialNodes[i].Node().Name)
280+
}
281+
klog.Infof("%v potential nodes for preemption, first %v are: %v", len(potentialNodes), len(sample), sample)
282+
}
276283
var pdbs []*policy.PodDisruptionBudget
277284
if g.pdbLister != nil {
278285
pdbs, err = g.pdbLister.List(labels.Everything())
@@ -1042,7 +1049,6 @@ func nodesWherePreemptionMightHelp(nodes []*framework.NodeInfo, fitErr *FitError
10421049
if fitErr.FilteredNodesStatuses[name].Code() == framework.UnschedulableAndUnresolvable {
10431050
continue
10441051
}
1045-
klog.V(3).Infof("Node %v is a potential node for preemption.", name)
10461052
potentialNodes = append(potentialNodes, node)
10471053
}
10481054
return potentialNodes

0 commit comments

Comments
 (0)