On 8/17/26 2:44 PM, Ales Musil wrote: > On Mon, Aug 17, 2026 at 2:22 PM Dumitru Ceara via dev < > [email protected]> wrote: > >> The test was incorrectly checking SB logical flows without waiting for >> northd to actually run and reconcile the contents of the SB with what >> had changed in the NB database. >> >> Hit in ovsrobot's CI: >> >> https://github.com/ovsrobot/ovn/actions/runs/32013870785/job/95339930296#step:13:5814 >> >> Fixes: 370527673c2b ("northd: Routing protocol port redirection.") >> Signed-off-by: Dumitru Ceara <[email protected]> >> --- >> tests/ovn-northd.at | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/tests/ovn-northd.at b/tests/ovn-northd.at >> index 2b9e95d7f1..8c8d7852e7 100644 >> --- a/tests/ovn-northd.at >> +++ b/tests/ovn-northd.at >> @@ -15225,6 +15225,8 @@ check ovn-nbctl ls-add ls -- \ >> check ovn-nbctl lsp-add ls lsp-bgp -- \ >> lsp-set-addresses lsp-bgp unknown >> >> +check ovn-nbctl --wait=sb sync >> + >> # Function that ensures that no redirect rules are installed. >> check_no_redirect() { >> AT_CHECK([ovn-sbctl dump-flows ls | grep ls_in_l2_lkup | grep -E >> "tcp.dst == 179|tcp.src == 179" | wc -l], [0], [0 >> -- >> 2.55.0 >> >> _______________________________________________ >> dev mailing list >> [email protected] >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> >> > Looks good to me, thanks. > Acked-by: Ales Musil <[email protected]> >
Thanks, Ales, for the review! Applied to main, 26.03 and 25.09. Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
