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
