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]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
