Re: [ovs-dev] [PATCH v2 10/15] test: tc does not support conntrack timeout, skip the related test.

[Roi Dayan via dev]

On 15/11/2022 14:32, Eelco Chaudron wrote:
> 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.at |    1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/tests/system-offloads.at b/tests/system-offloads.at
> index c8d1fd183..a9bfda164 100644
> --- a/tests/system-offloads.at
> +++ b/tests/system-offloads.at
> @@ -51,7 +51,6 @@ conntrack - IPv4 fragmentation with fragments specified
>  conntrack - IPv6 fragmentation + cvlan
>  conntrack - Fragmentation over vxlan
>  conntrack - IPv6 Fragmentation over vxlan
> -conntrack - zone-based timeout policy
>  conntrack - multiple zones, local
>  conntrack - multi-stage pipeline, local
>  conntrack - ICMP related with NAT
> 

the commit is missing the override macro for offloads
in system-offloads.at


+# Conntrack timeout not supported for tc.
+m4_define([CHECK_CONNTRACK_TIMEOUT],
+[
+     AT_SKIP_IF([:])
+])
+
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev