Hello, MJ,
Thanks for the patch!
The subject of this patch is a bit too generic, because there is a
myriad of tests doing dump-flows, and we only need to change this
specific one.
On 2/17/26 14:47, MJ Ponsonby via dev wrote:
This test is failing on s390x architectures due to the output being
ordered wrong [0]. This is a relatively easy fix and in other areas of
Should we mention which specific test needs updating instead of "this test"?
The reference to the GitHub Actions job is ephemeral, and while useful
to support review (and perhaps it belongs below the --- ?) we need
something more durable for the commit message.
the testsuite we do the same thing to ensure consistency, so it makes
sense to be done here.
[0]:
https://github.com/canonical/ubuntu-ovn-robot/actions/runs/22074516777/job/63786587659
Add a Reported-at: https://launchpad.net/bugs/2142024 here?
Signed-off-by: MJ Ponsonby <[email protected]>
---
FTR; I put your patch into a bespoke run on this architecture [1] and
can confirm it resolves the consistent failure we observe.
1:
https://github.com/canonical/ubuntu-ovn-robot/actions/runs/22105494561/job/63886895136#step:4:3916
--
Frode Nordahl
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
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
