Skip to content

Commit

Permalink
ovn: Make two end-to-end tests more reliable.
Browse files Browse the repository at this point in the history
These tests change the northbound configuration and then immediately check
that the changes have taken effect on the hypervisors.  This can't work
reliably, so add a sleep to each one.

Signed-off-by: Ben Pfaff <blp@ovn.org>
  • Loading branch information
blp committed Jul 27, 2016
1 parent ca2d5f7 commit 5d1ce90
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions tests/ovn.at
Expand Up @@ -912,6 +912,11 @@ done
# set address for lp13 with invalid characters.
# lp13 should be configured with only 192.168.0.13.
ovn-nbctl lsp-set-addresses lp13 "f0:00:00:00:00:13 192.168.0.13 invalid 192.169.0.13"

# Allow some time for ovn-northd and ovn-controller to catch up.
# XXX This should be more systematic.
sleep 1

sip=`ip_to_hex 192 168 0 11`
tip=`ip_to_hex 192 168 0 13`
test_arp 11 f00000000011 $sip $tip f00000000013
Expand Down Expand Up @@ -2494,6 +2499,10 @@ as hv1 ovs-ofctl dump-flows br-int
#Disable router R1
ovn-nbctl set Logical_Router R1 enabled=false

# Allow some time for ovn-northd and ovn-controller to catch up.
# XXX This should be more systematic.
sleep 1

echo "---------SB dump-----"
ovn-sbctl list datapath_binding
echo "---------------------"
Expand Down

0 comments on commit 5d1ce90

Please sign in to comment.