From 6420e7c39b634aa81889a28055b781e77ce89bf9 Mon Sep 17 00:00:00 2001 From: gray Date: Fri, 26 May 2023 12:10:24 +0800 Subject: [PATCH] test: delete ginkgo test "NodePort with L7 Policy from outside" The test case was introduced to cover issue #21954, but it turned out the test is buggy and caused a number of CI flakes (#25119). Consequently, PR #25236 put the test case under quarantine. This commit removes that problematic test, as the target scenario has been covered by connectivity test in cilium-cli (https://github.com/cilium/cilium-cli/pull/1547). Signed-off-by: Zhichuan Liang --- test/k8s/services.go | 23 ----------------------- 1 file changed, 23 deletions(-) diff --git a/test/k8s/services.go b/test/k8s/services.go index 5d1905aba2c72..0bd9c384362f8 100644 --- a/test/k8s/services.go +++ b/test/k8s/services.go @@ -656,29 +656,6 @@ Secondary Interface %s :: IPv4: (%s, %s), IPv6: (%s, %s)`, }) }) - SkipContextIf(func() bool { - return helpers.DoesNotRunWithKubeProxyReplacement() || helpers.RunsOnAKS() || helpers.DoesNotExistNodeWithoutCilium() || - helpers.SkipQuarantined() - }, "with L7 policy", func() { - var demoPolicyL7 string - - BeforeAll(func() { - demoPolicyL7 = helpers.ManifestGet(kubectl.BasePath(), "l7-policy-demo.yaml") - }) - - AfterAll(func() { - kubectl.Delete(demoPolicyL7) - // Same reason as in other L7 test above - kubectl.CiliumExecMustSucceedOnAll(context.TODO(), - "cilium bpf ct flush global", "Unable to flush CT maps") - }) - - It("Tests NodePort with L7 Policy from outside", func() { - applyPolicy(kubectl, demoPolicyL7) - testNodePort(kubectl, ni, false, true, 0) - }) - }) - It("ClusterIP cannot be accessed externally when access is disabled", func() { Expect(curlClusterIPFromExternalHost(kubectl, ni)).