upstream/kubernetes
1
0
Fork 0
mirror of https://github.com/kubernetes/kubernetes.git synced 2026-06-09 00:34:10 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

elaborate comment more

Browse source
This commit is contained in:
Kensei Nakada 2024-06-20 23:43:41 +00:00
parent fa8da84835
commit 2304806cbe
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
pkg/scheduler/internal/queue/scheduling_queue.go
View file

@ -1194,10 +1194,15 @@ func (p *PriorityQueue) movePodsToActiveOrBackoffQueue(logger klog.Logger, podIn
activated := false
for _, pInfo := range podInfoList {
// When handling events takes time, a scheduling throughput gets impacted negatively
// because of a shared lock within PriorityQueue, which Pop() also requires.
//
// Scheduling-gated Pods never get schedulable with any events,
// except the Pods themselves got updated, which isn't handled by movePodsToActiveOrBackoffQueue.
// So, here we skip them early here so that they don't go through isPodWorthRequeuing,
// So, we can skip them early here so that they don't go through isPodWorthRequeuing,
// which isn't fast enough when the number of scheduling-gated Pods in unschedulablePods is large.
// This is a hotfix, which might be changed
// once we have a better general solution for the shared lock issue.
//
// Note that we cannot skip all pInfo.Gated Pods here
// because PreEnqueue plugins apart from the scheduling gate plugin may change the gating status