Skip to content

Commit c60a7af

Browse files
committed
tests: Fix flaky "Routing protocol control plane redirect" test.
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: 3705276 ("northd: Routing protocol port redirection.") Acked-by: Ales Musil <amusil@redhat.com> Signed-off-by: Dumitru Ceara <dceara@redhat.com>
1 parent d79e6cd commit c60a7af

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

tests/ovn-northd.at

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15225,6 +15225,8 @@ check ovn-nbctl ls-add ls -- \
1522515225
check ovn-nbctl lsp-add ls lsp-bgp -- \
1522615226
lsp-set-addresses lsp-bgp unknown
1522715227

15228+
check ovn-nbctl --wait=sb sync
15229+
1522815230
# Function that ensures that no redirect rules are installed.
1522915231
check_no_redirect() {
1523015232
AT_CHECK([ovn-sbctl dump-flows ls | grep ls_in_l2_lkup | grep -E "tcp.dst == 179|tcp.src == 179" | wc -l], [0], [0

0 commit comments

Comments
 (0)