Skip to content

Commit

Permalink
test: Unquarantine K8sDatapathConfig Encapsulation
Browse files Browse the repository at this point in the history
The whole group of encapsulation-related tests was quarantined in
#18051. But there was never any
reason to quarantine the whole group as only a couple of tests are
actually failing.

According to DataStudio [1], nowadays only the iptables-masquerading
tests are failing. So this commit reenables all tests in that group
except for the two iptables-masquerading tests.

Note that this group includes our only coverage of IPsec + VXLAN.

1 - https://datastudio.google.com/s/nBvri2iDdrc
Signed-off-by: Paul Chaignon <paul@cilium.io>
  • Loading branch information
pchaigno committed Dec 12, 2022
1 parent 398cf5e commit 7212503
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/k8s/datapath_configuration.go
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ var _ = Describe("K8sDatapathConfig", func() {
})
})

SkipContextIf(helpers.SkipQuarantined, "Encapsulation", func() {
Context("Encapsulation", func() {
validateBPFTunnelMap := func() {
By("Checking that BPF tunnels are in place")
ciliumPod, err := kubectl.GetCiliumPodOnNode(helpers.K8s1)
Expand Down Expand Up @@ -216,7 +216,7 @@ var _ = Describe("K8sDatapathConfig", func() {
}
})

It("Check iptables masquerading with random-fully", func() {
SkipItIf(helpers.SkipQuarantined, "Check iptables masquerading with random-fully", func() {
options := map[string]string{
"bpf.masquerade": "false",
"enableIPv6Masquerade": "true",
Expand All @@ -233,7 +233,7 @@ var _ = Describe("K8sDatapathConfig", func() {
Should(BeTrue(), "IPv6 connectivity test to http://google.com failed")
})

It("Check iptables masquerading without random-fully", func() {
SkipItIf(helpers.SkipQuarantined, "Check iptables masquerading without random-fully", func() {
options := map[string]string{
"bpf.masquerade": "false",
"enableIPv6Masquerade": "true",
Expand Down

0 comments on commit 7212503

Please sign in to comment.