Skip to content

Commit

Permalink
test: reenable check for encryption and direct routing with bpf_host
Browse files Browse the repository at this point in the history
The test was disabled because of issue #12205: When bpf_host was loading
on the native device, the source identity of packet on the destination
node was resolved to WORLD and policy enforcement would fail.

This has now been fixed via #16136, and we can run the test again.

Also adjust the conditions for the test, to reflect the changes to
surrounding IPSec tests from f1209d0 ("test: Enable IPSec tests on
4.19").

Signed-off-by: Quentin Monnet <quentin@isovalent.com>
  • Loading branch information
qmonnet authored and aanm committed Jul 15, 2021
1 parent 23ffda0 commit d7eda2d
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions test/k8sT/DatapathConfiguration.go
Expand Up @@ -716,10 +716,7 @@ var _ = Describe("K8sDatapathConfig", func() {
Expect(testPodConnectivityAcrossNodes(kubectl)).Should(BeTrue(), "Connectivity test between nodes failed")
})

// This test is broken because of #12205. In short, when bpf_host is
// loading on the native device, the source identity of packet on the
// destination node is resolved to WORLD and policy enforcement fails.
XIt("Check connectivity with transparent encryption and direct routing with bpf_host", func() {
SkipItIf(helpers.RunsWithoutKubeProxy, "Check connectivity with transparent encryption and direct routing with bpf_host", func() {
privateIface, err := kubectl.GetPrivateIface()
Expect(err).Should(BeNil(), "Unable to determine the private interface")
defaultIface, err := kubectl.GetDefaultIface()
Expand All @@ -734,6 +731,7 @@ var _ = Describe("K8sDatapathConfig", func() {
"encryption.ipsec.interface": privateIface,
"devices": devices,
"hostFirewall": "false",
"kubeProxyReplacement": "disabled",
}, DeployCiliumOptionsAndDNS)
Expect(testPodConnectivityAcrossNodes(kubectl)).Should(BeTrue(), "Connectivity test between nodes failed")
})
Expand Down

0 comments on commit d7eda2d

Please sign in to comment.