Skip to content

Commit d81a5c6

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> (cherry picked from commit c60a7af)
1 parent 2314091 commit d81a5c6

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
@@ -14622,6 +14622,8 @@ check ovn-nbctl ls-add ls -- \
1462214622
check ovn-nbctl lsp-add ls lsp-bgp -- \
1462314623
lsp-set-addresses lsp-bgp unknown
1462414624

14625+
check ovn-nbctl --wait=sb sync
14626+
1462514627
# Function that ensures that no redirect rules are installed.
1462614628
check_no_redirect() {
1462714629
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)