Merge pull request #33133 from mml/omg-loop-variables

Automatic merge from submit-queue Stop referring to loop variable inside closure. Prior to this, I was actually running the same (last) test 5 times in a row. :-( Fixes #33137

Merge pull request #33133 from mml/omg-loop-variables
cfecc2fa · Kubernetes Submit Queue · GitHub · 0986a01f · f2ab87b6 · cfecc2fa
Commit cfecc2fa authored Sep 20, 2016 by Kubernetes Submit Queue Committed by GitHub Sep 20, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

disruption.go test/e2e/disruption.go +2 -1

No files found.
--- a/test/e2e/disruption.go
+++ b/test/e2e/disruption.go
@@ -113,7 +113,8 @@ var _ = framework.KubeDescribe("DisruptionController", func() {
 			shouldDeny:     true,
 		},
 	}
-	for _, c := range evictionCases {
+	for i := range evictionCases {
+		c := evictionCases[i]
 		expectation := "should allow an eviction"
 		if c.shouldDeny {
 			expectation = "should not allow an eviction"