Skip to content

Commit 9aa6d79

Browse files
authored
Merge pull request #1688 from ingvagabund/plugins-taints-do-not-list-all-pods
RemovePodsViolatingNodeTaints: list only pods that are not failed/suceeded
2 parents 7174626 + 6691720 commit 9aa6d79

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/framework/plugins/removepodsviolatingnodetaints/node_taint.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ func (d *RemovePodsViolatingNodeTaints) Name() string {
106106
func (d *RemovePodsViolatingNodeTaints) Deschedule(ctx context.Context, nodes []*v1.Node) *frameworktypes.Status {
107107
for _, node := range nodes {
108108
klog.V(1).InfoS("Processing node", "node", klog.KObj(node))
109-
pods, err := podutil.ListAllPodsOnANode(node.Name, d.handle.GetPodsAssignedToNodeFunc(), d.podFilter)
109+
pods, err := podutil.ListPodsOnANode(node.Name, d.handle.GetPodsAssignedToNodeFunc(), d.podFilter)
110110
if err != nil {
111111
// no pods evicted as error encountered retrieving evictable Pods
112112
return &frameworktypes.Status{

0 commit comments

Comments
 (0)