On 05/09/2022 16:07, Eelco Chaudron wrote:
> TC flower does not allow the push of the implicit null labels (RFC3032).
> Avoid the use of such labels in the MPLS test cases.
>
> Signed-off-by: Eelco Chaudron <[email protected]>
> ---
> tests/system-offloads-testsuite.at | 2 --
> tests/system-traffic.at | 8 ++++----
> 2 files changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/tests/system-offloads-testsuite.at
> b/tests/system-offloads-testsuite.at
> index 369860c80..43633af7e 100644
> --- a/tests/system-offloads-testsuite.at
> +++ b/tests/system-offloads-testsuite.at
> @@ -46,8 +46,6 @@ m4_define([OVS_TRAFFIC_VSWITCHD_START],
> # issue.
> m4_define([OVS_TEST_SKIP_LIST],
> [ovs_test_skip_list="
> -datapath - mpls actions
> -datapath - multiple mpls label pop
> datapath - basic truncate action
> datapath - truncate and output to gre tunnel by simulated packets
> datapath - truncate and output to gre tunnel
> diff --git a/tests/system-traffic.at b/tests/system-traffic.at
> index f7f885036..b911bd71c 100644
> --- a/tests/system-traffic.at
> +++ b/tests/system-traffic.at
> @@ -1167,8 +1167,8 @@ AT_CHECK([ovs-vsctl add-port br0 patch0])
> AT_CHECK([ovs-vsctl add-port br1 patch1])
>
> AT_DATA([flows.txt], [dnl
> -table=0,priority=100,dl_type=0x0800
> actions=push_mpls:0x8847,set_mpls_label:3,resubmit(,1)
> -table=0,priority=100,dl_type=0x8847,mpls_label=3
> actions=pop_mpls:0x0800,resubmit(,1)
> +table=0,priority=100,dl_type=0x0800
> actions=push_mpls:0x8847,set_mpls_label:4,resubmit(,1)
> +table=0,priority=100,dl_type=0x8847,mpls_label=4
> actions=pop_mpls:0x0800,resubmit(,1)
> table=0,priority=10 actions=resubmit(,1)
> table=1,priority=10 actions=normal
> ])
> @@ -1204,10 +1204,10 @@ AT_CHECK([ovs-vsctl add-port br0 patch0])
> AT_CHECK([ovs-vsctl add-port br1 patch1])
>
> AT_DATA([flows.txt], [dnl
> -table=0,priority=100,dl_type=0x0800
> actions=push_mpls:0x8847,set_mpls_label:3,push_mpls:0x8847,set_mpls_label:2,push_mpls:0x8847,set_mpls_label:1,resubmit(,3)
> +table=0,priority=100,dl_type=0x0800
> actions=push_mpls:0x8847,set_mpls_label:4,push_mpls:0x8847,set_mpls_label:2,push_mpls:0x8847,set_mpls_label:1,resubmit(,3)
> table=0,priority=100,dl_type=0x8847,mpls_label=1
> actions=pop_mpls:0x8847,resubmit(,1)
> table=1,priority=100,dl_type=0x8847,mpls_label=2
> actions=pop_mpls:0x8847,resubmit(,2)
> -table=2,priority=100,dl_type=0x8847,mpls_label=3
> actions=pop_mpls:0x0800,resubmit(,3)
> +table=2,priority=100,dl_type=0x8847,mpls_label=4
> actions=pop_mpls:0x0800,resubmit(,3)
> table=0,priority=10 actions=resubmit(,3)
> table=3,priority=10 actions=normal
> ])
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Acked-by: Roi Dayan <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
