Skip to content

Commit

Permalink
test: delete ginkgo test "NodePort with L7 Policy from outside"
Browse files Browse the repository at this point in the history
[ upstream commit eb5bf06 ]

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
(cilium/cilium-cli#1547).

Signed-off-by: Zhichuan Liang <[email protected]>
Signed-off-by: Tam Mach <[email protected]>
Signed-off-by: Michi Mutsuzaki <[email protected]>
  • Loading branch information
jschwinger233 authored and pchaigno committed Jun 12, 2023
1 parent e0a32df commit 6ab3895
Showing 1 changed file with 0 additions and 22 deletions.
22 changes: 0 additions & 22 deletions test/k8s/services.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)).
Expand Down

0 comments on commit 6ab3895

Please sign in to comment.