From 945aea462779c1144e9738a87527cdc077045122 Mon Sep 17 00:00:00 2001 From: Alex Katsman Date: Mon, 12 Sep 2022 15:08:42 +0200 Subject: [PATCH] Remove 'DisabledForLargeClusters' tag for LB tests We performed several internal tests at Google both on GKE and Kubernetes, and we believe it's safe to re-enable them for large clusters. --- test/e2e/network/loadbalancer.go | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/test/e2e/network/loadbalancer.go b/test/e2e/network/loadbalancer.go index 49ce7fbc1f7..2b0b02d9b52 100644 --- a/test/e2e/network/loadbalancer.go +++ b/test/e2e/network/loadbalancer.go @@ -772,9 +772,8 @@ var _ = common.SIGDescribe("LoadBalancers", func() { } }) - // TODO: Get rid of [DisabledForLargeClusters] tag when issue #56138 is fixed. // [LinuxOnly]: Windows does not support session affinity. - ginkgo.It("should have session affinity work for LoadBalancer service with ESIPP on [Slow] [DisabledForLargeClusters] [LinuxOnly]", func() { + ginkgo.It("should have session affinity work for LoadBalancer service with ESIPP on [Slow] [LinuxOnly]", func() { // L4 load balancer affinity `ClientIP` is not supported on AWS ELB. e2eskipper.SkipIfProviderIs("aws") @@ -784,9 +783,8 @@ var _ = common.SIGDescribe("LoadBalancers", func() { execAffinityTestForLBService(f, cs, svc) }) - // TODO: Get rid of [DisabledForLargeClusters] tag when issue #56138 is fixed. // [LinuxOnly]: Windows does not support session affinity. - ginkgo.It("should be able to switch session affinity for LoadBalancer service with ESIPP on [Slow] [DisabledForLargeClusters] [LinuxOnly]", func() { + ginkgo.It("should be able to switch session affinity for LoadBalancer service with ESIPP on [Slow] [LinuxOnly]", func() { // L4 load balancer affinity `ClientIP` is not supported on AWS ELB. e2eskipper.SkipIfProviderIs("aws") @@ -796,9 +794,8 @@ var _ = common.SIGDescribe("LoadBalancers", func() { execAffinityTestForLBServiceWithTransition(f, cs, svc) }) - // TODO: Get rid of [DisabledForLargeClusters] tag when issue #56138 is fixed. // [LinuxOnly]: Windows does not support session affinity. - ginkgo.It("should have session affinity work for LoadBalancer service with ESIPP off [Slow] [DisabledForLargeClusters] [LinuxOnly]", func() { + ginkgo.It("should have session affinity work for LoadBalancer service with ESIPP off [Slow] [LinuxOnly]", func() { // L4 load balancer affinity `ClientIP` is not supported on AWS ELB. e2eskipper.SkipIfProviderIs("aws") @@ -808,9 +805,8 @@ var _ = common.SIGDescribe("LoadBalancers", func() { execAffinityTestForLBService(f, cs, svc) }) - // TODO: Get rid of [DisabledForLargeClusters] tag when issue #56138 is fixed. // [LinuxOnly]: Windows does not support session affinity. - ginkgo.It("should be able to switch session affinity for LoadBalancer service with ESIPP off [Slow] [DisabledForLargeClusters] [LinuxOnly]", func() { + ginkgo.It("should be able to switch session affinity for LoadBalancer service with ESIPP off [Slow] [LinuxOnly]", func() { // L4 load balancer affinity `ClientIP` is not supported on AWS ELB. e2eskipper.SkipIfProviderIs("aws")