diff mbox series

[ovs-dev,1/5] tests: fixed another flake in "send gratuitous ARP for NAT rules on HA distributed router"

Message ID 20230718221152.81282-2-xsimonar@redhat.com
State Accepted
Headers show
Series Unit and System tests fixes | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes success github build: passed

Commit Message

Xavier Simonart July 18, 2023, 10:11 p.m. UTC
Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
---
 tests/ovn.at | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/tests/ovn.at b/tests/ovn.at
index f2148faac..496403f36 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -13874,6 +13874,9 @@  AT_CHECK([grep $garp hv3_br_phys_tx | sort], [0], [])
 
 # at this point, we invert the priority of the gw chassis between hv2 and hv3
 
+# Reset hv2/br-phys_n1 before inverting gw chassis, as otherwise packet might be reset (while not resetting hv1/snoopvif
+# because not yet sent)
+as hv2 reset_pcap_file br-phys_n1 hv2/br-phys_n1
 ovn-nbctl --wait=hv \
           --id=@gc0 create Gateway_Chassis \
                     name=outside_gw1 chassis_name=hv2 priority=1 -- \
@@ -13881,7 +13884,6 @@  ovn-nbctl --wait=hv \
                     name=outside_gw2 chassis_name=hv3 priority=10 -- \
           set Logical_Router_Port lrp0 'gateway_chassis=[@gc0,@gc1]'
 
-as hv2 reset_pcap_file br-phys_n1 hv2/br-phys_n1
 as hv3 reset_pcap_file br-phys_n1 hv3/br-phys_n1
 as hv1 reset_pcap_file snoopvif hv1/snoopvif