Merge pull request kubernetes#93126 from gaurav1086/getCSINodeLimits_fix_range_issue

k8s-ci-robot · web-flow · commit b8ecff28d303 · 2020-10-19T18:20:07.000-07:00
[e2e/storage] fix range iterator copy issue in getCSINodeLimits
diff --git a/test/e2e/storage/testsuites/volumelimits.go b/test/e2e/storage/testsuites/volumelimits.go
@@ -349,9 +349,9 @@ func getCSINodeLimits(cs clientset.Interface, config *PerTestConfig, nodeName st
 			return false, nil
 		}
 		var csiDriver *storagev1.CSINodeDriver
-		for _, c := range csiNode.Spec.Drivers {
+		for i, c := range csiNode.Spec.Drivers {
 			if c.Name == driverInfo.Name || c.Name == config.GetUniqueDriverName() {
-				csiDriver = &c
+				csiDriver = &csiNode.Spec.Drivers[i]
 				break
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -349,9 +349,9 @@ func getCSINodeLimits(cs clientset.Interface, config *PerTestConfig, nodeName st`
`349`	`349`	`return false, nil`
`350`	`350`	`}`
`351`	`351`	`var csiDriver *storagev1.CSINodeDriver`
`352`		`- for _, c := range csiNode.Spec.Drivers {`
	`352`	`+ for i, c := range csiNode.Spec.Drivers {`
`353`	`353`	`if c.Name == driverInfo.Name \|\| c.Name == config.GetUniqueDriverName() {`
`354`		`- csiDriver = &c`
	`354`	`+ csiDriver = &csiNode.Spec.Drivers[i]`
`355`	`355`	`break`
`356`	`356`	`}`
`357`	`357`	`}`