From 6691720da5c406507d376574d37676f136e644e4 Mon Sep 17 00:00:00 2001 From: Jan Chaloupka Date: Sat, 10 May 2025 21:12:02 +0200 Subject: [PATCH] RemovePodsViolatingNodeTaints: list only pods that are not failed/suceeded Listing pods was incorrectly changed to listing all pods during code refactoring. --- .../plugins/removepodsviolatingnodetaints/node_taint.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/framework/plugins/removepodsviolatingnodetaints/node_taint.go b/pkg/framework/plugins/removepodsviolatingnodetaints/node_taint.go index e447a4662..a336f1c70 100644 --- a/pkg/framework/plugins/removepodsviolatingnodetaints/node_taint.go +++ b/pkg/framework/plugins/removepodsviolatingnodetaints/node_taint.go @@ -106,7 +106,7 @@ func (d *RemovePodsViolatingNodeTaints) Name() string { func (d *RemovePodsViolatingNodeTaints) Deschedule(ctx context.Context, nodes []*v1.Node) *frameworktypes.Status { for _, node := range nodes { klog.V(1).InfoS("Processing node", "node", klog.KObj(node)) - pods, err := podutil.ListAllPodsOnANode(node.Name, d.handle.GetPodsAssignedToNodeFunc(), d.podFilter) + pods, err := podutil.ListPodsOnANode(node.Name, d.handle.GetPodsAssignedToNodeFunc(), d.podFilter) if err != nil { // no pods evicted as error encountered retrieving evictable Pods return &frameworktypes.Status{