Skip to content

Commit

Permalink
[ipfwd/test_nhop_group]: Support multi-asic in interface flap test (s…
Browse files Browse the repository at this point in the history
…onic-net#15486)

Arp eviction commands need to respect the asic namespace when being applied to multi-asic devices, as the procfs entries are for each individual asic.

Signed-off-by: Liam Kearney <liamkearney@microsoft.com>
  • Loading branch information
liamkearney-msft authored and mssonicbld committed Nov 27, 2024
1 parent 4a5d341 commit 4a6307f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/ipfwd/test_nhop_group.py
Original file line number Diff line number Diff line change
Expand Up @@ -867,7 +867,7 @@ def test_nhop_group_interface_flap(duthosts, enum_rand_one_per_hwsku_frontend_ho

# Enable kernel flag to not evict ARP entries when the interface goes down
# and shut the fanout switch ports.
duthost.shell(arp_noevict_cmd % gather_facts['src_router_intf_name'])
asic.command(arp_noevict_cmd % gather_facts['src_router_intf_name'])
for i in range(0, len(gather_facts['src_port'])):
fanout, fanout_port = fanout_switch_port_lookup(fanouthosts, duthost.hostname,
gather_facts['src_port'][i])
Expand Down Expand Up @@ -907,6 +907,6 @@ def test_nhop_group_interface_flap(duthosts, enum_rand_one_per_hwsku_frontend_ho
logger.info("portstats: %s", result['stdout'])

finally:
duthost.shell(arp_evict_cmd % gather_facts['src_router_intf_name'])
asic.command(arp_evict_cmd % gather_facts['src_router_intf_name'])
nhop.delete_routes()
arplist.clean_up()

0 comments on commit 4a6307f

Please sign in to comment.