github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #105205 from ingvagabund/do-not-reference-control-variable-in-for

Browse Source 
e2e scheduling priorities: do not reference control loop variable
This commit is contained in:
Kubernetes Prow Robot
2021-09-23 08:37:22 -07:00
committed by GitHub
parent 2541fcf256 b3249a1b39
commit 6c2f644482
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
test/e2e/scheduling/priorities.go
View File

@@ -485,9 +485,9 @@ func podListForEachNode(cs clientset.Interface) map[string][]*v1.Pod {
if err != nil { if err != nil {
framework.Failf("Expect error of invalid, got : %v", err) framework.Failf("Expect error of invalid, got : %v", err)
} }
for _, pod := range allPods.Items { for i, pod := range allPods.Items {
nodeName := pod.Spec.NodeName nodeName := pod.Spec.NodeName
nodeNameToPodList[nodeName] = append(nodeNameToPodList[nodeName], &pod) nodeNameToPodList[nodeName] = append(nodeNameToPodList[nodeName], &allPods.Items[i])
} }
return nodeNameToPodList return nodeNameToPodList
} }