Skip to content

Commit 296a16e

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 4835ae1 commit 296a16e

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
@@ -14988,6 +14988,8 @@ check ovn-nbctl ls-add ls -- \
1498814988
check ovn-nbctl lsp-add ls lsp-bgp -- \
1498914989
lsp-set-addresses lsp-bgp unknown
1499014990

14991+
check ovn-nbctl --wait=sb sync
14992+
1499114993
# Function that ensures that no redirect rules are installed.
1499214994
check_no_redirect() {
1499314995
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)