The tc conntrack implementation does not support the timeout option.
The current implementation is silently ignoring the timeout option
by adding a general conntrack entry.

This patch will skip the related test by overriding the support macro.

Signed-off-by: Eelco Chaudron <echau...@redhat.com>
Acked-by: Roi Dayan <r...@nvidia.com>
---
 tests/system-offloads-testsuite-macros.at |    6 ++++++
 tests/system-traffic.at                   |    1 -
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/tests/system-offloads-testsuite-macros.at 
b/tests/system-offloads-testsuite-macros.at
index 5d7044f42..322166b8c 100644
--- a/tests/system-offloads-testsuite-macros.at
+++ b/tests/system-offloads-testsuite-macros.at
@@ -40,3 +40,9 @@ m4_define([CHECK_CONNTRACK_ALG],
 [
      AT_SKIP_IF([:])
 ])
+
+# Conntrack timeout not supported for tc.
+m4_define([CHECK_CONNTRACK_TIMEOUT],
+[
+     AT_SKIP_IF([:])
+])
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index 05e0473ec..eb03c69be 100644
--- a/tests/system-traffic.at
+++ b/tests/system-traffic.at
@@ -4333,7 +4333,6 @@ AT_CLEANUP
 AT_SETUP([conntrack - zone-based timeout policy])
 CHECK_CONNTRACK()
 CHECK_CONNTRACK_TIMEOUT()
-CHECK_NO_TC_OFFLOAD()
 OVS_TRAFFIC_VSWITCHD_START()
 
 ADD_NAMESPACES(at_ns0, at_ns1)

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to