Skip to content

Commit abede62

Browse files
authored
Merge pull request kubernetes#93857 from soltysh/fix_lb_test
Skip LB test per schedulable node count
2 parents fa13dc1 + 5765bf7 commit abede62

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

test/e2e/apps/deployment.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ import (
4444
deploymentutil "k8s.io/kubernetes/pkg/controller/deployment/util"
4545
"k8s.io/kubernetes/test/e2e/framework"
4646
e2edeployment "k8s.io/kubernetes/test/e2e/framework/deployment"
47+
e2enode "k8s.io/kubernetes/test/e2e/framework/node"
4748
e2epod "k8s.io/kubernetes/test/e2e/framework/pod"
4849
e2ereplicaset "k8s.io/kubernetes/test/e2e/framework/replicaset"
4950
e2eresource "k8s.io/kubernetes/test/e2e/framework/resource"
@@ -135,6 +136,9 @@ var _ = SIGDescribe("Deployment", func() {
135136
})
136137
ginkgo.It("should not disrupt a cloud load-balancer's connectivity during rollout", func() {
137138
e2eskipper.SkipUnlessProviderIs("aws", "azure", "gce", "gke")
139+
nodes, err := e2enode.GetReadySchedulableNodes(c)
140+
framework.ExpectNoError(err)
141+
e2eskipper.SkipUnlessAtLeast(len(nodes.Items), 3, "load-balancer test requires at least 3 schedulable nodes")
138142
testRollingUpdateDeploymentWithLocalTrafficLoadBalancer(f)
139143
})
140144
// TODO: add tests that cover deployment.Spec.MinReadySeconds once we solved clock-skew issues

0 commit comments

Comments
 (0)