From 1685f4d56886bb5314bd375b1cbb169de10f15bd Mon Sep 17 00:00:00 2001 From: Grant Spence Date: Wed, 29 Jul 2026 19:48:53 -0400 Subject: [PATCH] OCPBUGS-63219: Remove NLB hairpin workaround from dual-stack test The manual target-group-attributes annotation to disable client IP preservation is no longer needed. New NLB IngressControllers default to PROXY protocol (openshift/cluster-ingress-operator#1426), which fixes hairpin connection failures without manual intervention. Co-Authored-By: Claude Opus 4.6 (1M context) --- test/extended/router/dualstack.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/test/extended/router/dualstack.go b/test/extended/router/dualstack.go index 74b2721b5ac1..a8f3f36b94a2 100644 --- a/test/extended/router/dualstack.go +++ b/test/extended/router/dualstack.go @@ -79,12 +79,6 @@ var _ = g.Describe("[sig-network-edge][OCPFeatureGate:AWSDualStackInstall][Featu }() o.Expect(err).NotTo(o.HaveOccurred(), "new router shard did not rollout") - g.By("Disabling client IP preservation on the NLB target group to avoid hairpin issues (OCPBUGS-63219)") - routerSvcName := "router-" + shardIngressCtrl.Name - err = oc.AsAdmin().Run("annotate").Args("service", "-n", "openshift-ingress", routerSvcName, - "service.beta.kubernetes.io/aws-load-balancer-target-group-attributes=preserve_client_ip.enabled=false").Execute() - o.Expect(err).NotTo(o.HaveOccurred()) - g.By("Labelling the namespace for the shard") err = oc.AsAdmin().Run("label").Args("namespace", oc.Namespace(), "type="+oc.Namespace()).Execute() o.Expect(err).NotTo(o.HaveOccurred())