From 6ab38950f33d12c4c71354e20a9e48f7b4a46db3 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" [ upstream commit eb5bf069b04720d3539ecfc1936bf65760bf0857 ] 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 Signed-off-by: Tam Mach Signed-off-by: Michi Mutsuzaki --- test/k8s/services.go | 22 ---------------------- 1 file changed, 22 deletions(-) diff --git a/test/k8s/services.go b/test/k8s/services.go index e08e0dee79fdad..e7bbb645fe1beb 100644 --- a/test/k8s/services.go +++ b/test/k8s/services.go @@ -609,28 +609,6 @@ Secondary Interface %s :: IPv4: (%s, %s), IPv6: (%s, %s)`, }) }) - SkipContextIf(func() bool { - return helpers.DoesNotRunWithKubeProxyReplacement() || helpers.RunsOnAKS() || helpers.DoesNotExistNodeWithoutCilium() - }, "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)).