On Tue, Feb 17, 2026 at 7:52 PM MJ Ponsonby via dev <[email protected]>
wrote:

> The test `ovn-controller - I-P for address set update: multiple ASes
> used by same lflow` is failing on s390x architectures due to the output
> being ordered wrong. This is a relatively easy fix and in other areas of
> the testsuite we do the same thing to ensure consistency, so it makes
> sense to be done here.
>
> Reported-at: https://launchpad.net/bugs/2142024
> Signed-off-by: MJ Ponsonby <[email protected]>
> ---
>

Hi MJ,

thank you for the fix. After a quick search I have found
few more places where the sort is missing too, would you
mind changing those too? In some cases there is just a single
flow, but as you mentioned in the commit message, it's still a
good to be consistent.



>  tests/ovn-controller.at | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/ovn-controller.at b/tests/ovn-controller.at
> index 181a8afe0..c98de9bc4 100644
> --- a/tests/ovn-controller.at
> +++ b/tests/ovn-controller.at
> @@ -1522,7 +1522,7 @@ for i in $(seq 10); do
>      check ovn-nbctl --wait=hv sync
>      if test "$i" = 9; then
>          AT_CHECK_UNQUOTED([ovs-ofctl dump-flows br-int
> table=$acl_eval,reg15=0x$port_key | \
> -            grep -v reply | awk '{print $7, $8}'], [0], [dnl
> +            grep -v reply | awk '{print $7, $8}' | sort], [0], [dnl
>  
> priority=1100,ip,reg15=0x$port_key,metadata=0x$dp_key,nw_src=10.0.0.10,nw_dst=10.0.0.6
> actions=load:0x1->OXM_OF_PKT_REG4[[49]],resubmit(,$acl_sample)
>  
> priority=1100,ip,reg15=0x$port_key,metadata=0x$dp_key,nw_src=10.0.0.10,nw_dst=10.0.0.7
> actions=load:0x1->OXM_OF_PKT_REG4[[49]],resubmit(,$acl_sample)
>  ])
> --
> 2.43.0
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
Regards,
Ales
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to