From bac8086b61c170575684c1ab4219ff3427192737 Mon Sep 17 00:00:00 2001 From: Lan Luo Date: Fri, 26 Apr 2024 17:27:37 +0800 Subject: [PATCH] Skip IPSec/WireGuard e2e test when the Multicast feature is enabled This is an e2e fix because of PR #5920. Signed-off-by: Lan Luo --- test/e2e/fixtures.go | 10 ++++++++++ test/e2e/ipsec_test.go | 1 + test/e2e/wireguard_test.go | 1 + 3 files changed, 12 insertions(+) diff --git a/test/e2e/fixtures.go b/test/e2e/fixtures.go index 71fa872759e..8abfddcb17a 100644 --- a/test/e2e/fixtures.go +++ b/test/e2e/fixtures.go @@ -159,6 +159,16 @@ func skipIfNoVMs(tb testing.TB) { } } +func skipIfMulticastEnabled(tb testing.TB, data *TestData) { + agentConf, err := data.GetAntreaAgentConf() + if err != nil { + tb.Fatalf("Error getting option multicast.enable value") + } + if agentConf.Multicast.Enable { + tb.Skipf("Skipping test because option multicast.enable is true") + } +} + func skipIfFeatureDisabled(tb testing.TB, feature featuregate.Feature, checkAgent bool, checkController bool) { if checkAgent { if featureGate, err := GetAgentFeatures(); err != nil { diff --git a/test/e2e/ipsec_test.go b/test/e2e/ipsec_test.go index b972f18ecdf..931b8e49f86 100644 --- a/test/e2e/ipsec_test.go +++ b/test/e2e/ipsec_test.go @@ -42,6 +42,7 @@ func TestIPSec(t *testing.T) { t.Fatalf("Error when setting up test: %v", err) } defer teardownTest(t, data) + skipIfMulticastEnabled(t, data) t.Logf("Redeploy Antrea with IPsec tunnel enabled") data.redeployAntrea(t, deployAntreaIPsec) diff --git a/test/e2e/wireguard_test.go b/test/e2e/wireguard_test.go index 89c86b83dbe..afb73cae6b3 100644 --- a/test/e2e/wireguard_test.go +++ b/test/e2e/wireguard_test.go @@ -40,6 +40,7 @@ func TestWireGuard(t *testing.T) { t.Fatalf("Error when setting up test: %v", err) } defer teardownTest(t, data) + skipIfMulticastEnabled(t, data) skipIfEncapModeIsNot(t, data, config.TrafficEncapModeEncap) for _, node := range clusterInfo.nodes { skipIfMissingKernelModule(t, data, node.name, []string{"wireguard"})