Skip to content

Commit

Permalink
tests: fixed "send gratuitous ARP for NAT rules on HA distributed rou…
Browse files Browse the repository at this point in the history
…ter"

Fixes: 0c89c0f ("tests: fixed another flake in "send gratuitous ARP for NAT rules on HA distributed router"")

Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
Acked-by: Mark Michelson <mmichels@redhat.com>
Signed-off-by: Dumitru Ceara <dceara@redhat.com>
  • Loading branch information
simonartxavier authored and dceara committed Oct 6, 2023
1 parent f8df558 commit cbe3594
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions tests/ovn.at
Original file line number Diff line number Diff line change
Expand Up @@ -13868,16 +13868,18 @@ 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 packets 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 -- \
--id=@gc1 create Gateway_Chassis \
name=outside_gw2 chassis_name=hv3 priority=10 -- \
set Logical_Router_Port lrp0 'gateway_chassis=[@gc0,@gc1]'

# We expect not to receive garp on hv2 after inverting the priority.
# Hence reset hv2 after inverting priority as otherwise a garp might
# be received on hv2 between the reset and the priority change.

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

Expand Down

0 comments on commit cbe3594

Please sign in to comment.