Merge pull request #18843 from jpeeler/hostpath-pass-39

derekwaynecarr · web-flow · commit 9b766ba9a29e · 2018-03-05T17:23:50.000-05:00
Allow all nodes to run upstream kube tests
diff --git a/test/extended/util/test.go b/test/extended/util/test.go
@@ -239,6 +239,9 @@ func createTestingNS(baseName string, c kclientset.Interface, labels map[string]
 			return ns, err
 		}
 		addRoleToE2EServiceAccounts(authorizationClient, []kapiv1.Namespace{*ns}, bootstrappolicy.ViewRoleName)
+
+		// in practice too many kube tests ignore scheduling constraints
+		allowAllNodeScheduling(c, ns.Name)
 	}
 
 	// some tests assume they can schedule to all nodes

Original file line number	Diff line number	Diff line change
`@@ -239,6 +239,9 @@ func createTestingNS(baseName string, c kclientset.Interface, labels map[string]`
`239`	`239`	`return ns, err`
`240`	`240`	`}`
`241`	`241`	`addRoleToE2EServiceAccounts(authorizationClient, []kapiv1.Namespace{*ns}, bootstrappolicy.ViewRoleName)`
	`242`	`+`
	`243`	`+ // in practice too many kube tests ignore scheduling constraints`
	`244`	`+ allowAllNodeScheduling(c, ns.Name)`
`242`	`245`	`}`
`243`	`246`
`244`	`247`	`// some tests assume they can schedule to all nodes`