From d28c8cd48879e31486bba3a5e91c32fcc5b2c7c6 Mon Sep 17 00:00:00 2001 From: Kensei Nakada Date: Wed, 7 May 2025 14:08:23 +0200 Subject: [PATCH] fix: not removing the plugin from the unsched plugins after PreEnqueue --- pkg/scheduler/backend/queue/scheduling_queue.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/pkg/scheduler/backend/queue/scheduling_queue.go b/pkg/scheduler/backend/queue/scheduling_queue.go index 284ae25ef92..84a63f4768d 100644 --- a/pkg/scheduler/backend/queue/scheduling_queue.go +++ b/pkg/scheduler/backend/queue/scheduling_queue.go @@ -579,9 +579,7 @@ func (p *PriorityQueue) runPreEnqueuePlugin(ctx context.Context, logger klog.Log p.metricsRecorder.ObservePluginDurationAsync(preEnqueue, pl.Name(), s.Code().String(), p.clock.Since(startTime).Seconds()) } if s.IsSuccess() { - // This plugin passed, remove it from the unschedulable plugins. // No need to change GatingPlugin; it's overwritten by the next PreEnqueue plugin if they gate this pod, or it's overwritten with an empty string if all PreEnqueue plugins pass. - pInfo.UnschedulablePlugins.Delete(pl.Name()) return s } pInfo.UnschedulablePlugins.Insert(pl.Name())