Skip to content

Commit

Permalink
tests: quarantine services nodeport w/ L7 policy test.
Browse files Browse the repository at this point in the history
Changes in c1a0dba may have introduced flakes for this set of tests
when attempting to reach NodePort from outside the cluster.

A replacement test is being added to cilium cli connectivity tests:

cilium/cilium-cli#1547

Quarantine this test for now until we can remove it.

Addresses: cilium#25119

Signed-off-by: Tom Hadlaw <[email protected]>
  • Loading branch information
tommyp1ckles committed May 2, 2023
1 parent 95fb5c3 commit f1ed7d7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/k8s/services.go
Original file line number Diff line number Diff line change
Expand Up @@ -645,7 +645,8 @@ Secondary Interface %s :: IPv4: (%s, %s), IPv6: (%s, %s)`,
})

SkipContextIf(func() bool {
return helpers.DoesNotRunWithKubeProxyReplacement() || helpers.RunsOnAKS() || helpers.DoesNotExistNodeWithoutCilium()
return helpers.DoesNotRunWithKubeProxyReplacement() || helpers.RunsOnAKS() || helpers.DoesNotExistNodeWithoutCilium() ||
helpers.SkipQuarantined()
}, "with L7 policy", func() {
var demoPolicyL7 string

Expand Down

0 comments on commit f1ed7d7

Please sign in to comment.