Merge pull request #55275 from shyamjvs/skip-esipp-slow-tests-on-large-cluster

Automatic merge from submit-queue (batch tested with PRs 54177, 55203, 55120, 55275, 55260). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Skip ESIPP [Slow] suite of networking tests for huge clusters Ref https://github.com/kubernetes/kubernetes/issues/52495#issuecomment-342523340 /cc @MrHohn @kubernetes/sig-network-misc

Merge pull request #55275 from shyamjvs/skip-esipp-slow-tests-on-large-cluster
f92f5449 · Kubernetes Submit Queue · GitHub · a3e00dbd · 9356c787 · f92f5449
Unverified Commit f92f5449 authored Nov 08, 2017 by Kubernetes Submit Queue Committed by GitHub Nov 08, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

service.go test/e2e/network/service.go +4 -0

No files found.
--- a/test/e2e/network/service.go
+++ b/test/e2e/network/service.go
@@ -1450,6 +1450,10 @@ var _ = SIGDescribe("ESIPP [Slow]", func() {
 		// requires cloud load-balancer support - this feature currently supported only on GCE/GKE
 		framework.SkipUnlessProviderIs("gce", "gke")

+		// Skipping this test for too large clusters due to issue #52495.
+		// TODO(MrHohn): Get rid of this when gce-side load-balancer improvements are done.
+		framework.SkipUnlessNodeCountIsAtMost(framework.GCPMaxInstancesInInstanceGroup)
+
 		cs = f.ClientSet
 		if nodes := framework.GetReadySchedulableNodesOrDie(cs); len(nodes.Items) > framework.LargeClusterMinNodesNumber {
 			loadBalancerCreateTimeout = framework.LoadBalancerCreateTimeoutLarge