Skip to content

Commit

Permalink
tests: Fix flaky "ovs-appctl -t ovsdb-server ...".
Browse files Browse the repository at this point in the history
Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
Signed-off-by: 0-day Robot <robot@bytheb.org>
  • Loading branch information
simonartxavier authored and ovsrobot committed Dec 10, 2024
1 parent e38def1 commit bc5b46e
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions tests/ovn-northd.at
Original file line number Diff line number Diff line change
Expand Up @@ -918,6 +918,10 @@ check as ovn-nb ovs-appctl -t ovsdb-server ovsdb-server/remove-remote "$conn"
conn2=punix:`pwd`/special.sock
check as ovn-nb ovs-appctl -t ovsdb-server ovsdb-server/add-remote "$conn2"

# The reply of previous unixctl does not guarantee that ovsdb-server added the remote.
# By running another unixctl command, we ensure that ovsdb-server had reconfigured the remotes.
AT_CHECK([test 1 = $(as ovn-nb ovs-appctl -t ovsdb-server ovsdb-server/list-remotes | grep -c special.sock)])

# ovn-northd won't respond to changes (because the nbdb connection dropped).
check ovn-nbctl --db="${conn2#p}" ls-add sw1
sleep 5
Expand Down

0 comments on commit bc5b46e

Please sign in to comment.