The test case "76: ovn -- 4 HV, 1 LS, 1 LR, packet test with HA
distributed router gateway port" fails sometimes. This patch fixes
this by leveraging the command ovn-nbctl sync. It also adds a
delay before sending a packet between foo1 and outside1, so it does
not get out of order with the gratuitous arp broadcast.
The failed test would look like this:
<snip>
checking packets in ext1/vif1-tx.pcap against ext1-vif1.expected:
ovn.at:12: waiting until $PYTHON "$ovs_srcdir/utilities/ovs-pcap.in"
$rcv_pcap > $rcv_text
rcv_n=`wc -l < "$rcv_text"`
echo "rcv_n=$rcv_n exp_n=$exp_n"
test $rcv_n -ge $exp_n...
<snip>
ovn.at:12: wait succeeded after 2 seconds
./ovn.at:8865: sort $rcv_text
expout 2020-04-22 15:24:29.948703642 -0400
/home/ffernand/work/ovn/tests/testsuite.dir/at-groups/76/stdout
2020-04-22 15:24:29.948703642 -0400
-1,2 +1,2
-f0000001020400000201020308004500001c000000003f110100c0a80102ac1001030035111100080000
+ffffffffffff00000201020308060001080006040001000002010203ac100101000000000000ac100101
ffffffffffff00000201020308060001080006040001000002010203ac100101000000000000ac100101
76. ovn.at:8695: 76. ovn -- 4 HV, 1 LS, 1 LR, packet test with HA distributed
router gateway port (ovn.at:8695): FAILED (ovn.at:8865)
Reported-by: Dumitru Ceara <[email protected]>
Signed-off-by: Flavio Fernandes <[email protected]>
---
tests/ovn.at | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index 2a7ee7528..149345101 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -8818,8 +8818,7 @@ as ext1 ovs-vsctl set open .
external-ids:ovn-bridge-mappings=phys:br-phys
AT_CHECK([ovn-nbctl --timeout=3 --wait=sb sync], [0], [ignore])
# Allow some time for ovn-northd and ovn-controller to catch up.
-# XXX This should be more systematic.
-sleep 2
+ovn-nbctl --wait=hv sync
ip_to_hex() {
printf "%02x%02x%02x%02x" "$@"
@@ -8886,6 +8885,8 @@ grep actions=mod_dl_dst:f0:00:00:01:02:04 | wc -l` -eq 1
fi
as ext1 reset_pcap_file ext1-vif1 ext1/vif1
+ sleep 1
+
# Resend packet from foo1 to outside1
as hv1 ovs-appctl netdev-dummy/receive hv1-vif1 $packet
--
2.17.1
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
