From d6b35eaed66ecea4170efec43029fbc1012c4a3c Mon Sep 17 00:00:00 2001 From: Jan Chaloupka Date: Mon, 15 Dec 2025 16:40:41 +0100 Subject: [PATCH] refactor(node_taint_test): apply pod single creation convention for p3 --- .../removepodsviolatingnodetaints/node_taint_test.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/pkg/framework/plugins/removepodsviolatingnodetaints/node_taint_test.go b/pkg/framework/plugins/removepodsviolatingnodetaints/node_taint_test.go index 35808b592..14d610cee 100644 --- a/pkg/framework/plugins/removepodsviolatingnodetaints/node_taint_test.go +++ b/pkg/framework/plugins/removepodsviolatingnodetaints/node_taint_test.go @@ -130,7 +130,10 @@ func TestDeletePodsViolatingNodeTaints(t *testing.T) { p2 := buildTestPod("p2", nodeName1, func(pod *v1.Pod) { pod.ObjectMeta.OwnerReferences = test.GetNormalPodOwnerRefList() }) - p3 := buildTestPod("p3", nodeName1, nil) + p3 := buildTestPod("p3", nodeName1, func(pod *v1.Pod) { + pod.ObjectMeta.OwnerReferences = test.GetNormalPodOwnerRefList() + addTolerationToPod(pod, "testTaint", "test", 1, v1.TaintEffectNoSchedule) + }) p4 := buildTestPod("p4", nodeName1, nil) p5 := buildTestPod("p5", nodeName1, nil) p6 := buildTestPod("p6", nodeName1, nil) @@ -140,7 +143,6 @@ func TestDeletePodsViolatingNodeTaints(t *testing.T) { p10 := buildTestPod("p10", nodeName2, nil) p11 := buildTestPod("p11", nodeName2, nil) p12 := buildTestPod("p11", nodeName2, nil) - p3.ObjectMeta.OwnerReferences = test.GetNormalPodOwnerRefList() p4.ObjectMeta.OwnerReferences = test.GetNormalPodOwnerRefList() p5.ObjectMeta.OwnerReferences = test.GetNormalPodOwnerRefList() p6.ObjectMeta.OwnerReferences = test.GetNormalPodOwnerRefList() @@ -176,7 +178,6 @@ func TestDeletePodsViolatingNodeTaints(t *testing.T) { // A Mirror Pod. p10.Annotations = test.GetMirrorPodAnnotation() - p3 = addTolerationToPod(p3, "testTaint", "test", 1, v1.TaintEffectNoSchedule) p4 = addTolerationToPod(p4, "testTaintX", "testX", 1, v1.TaintEffectNoSchedule) p12.Spec.NodeSelector = map[string]string{