Skip to content

Commit

Permalink
Extend config reload wait time for static route tests (#3749)
Browse files Browse the repository at this point in the history
*The arp_update script updates unresolved neighbors every 300 seconds. As the default config reload wait time is 120 seconds, the arp_update script may not finish the neighbor resolution before the config reload wait time and cause the static route test to fail. This PR aims to fix the timing issue for static route tests.
*Fix: Extend config reload wait time for static route tests
  • Loading branch information
shi-su authored Jul 9, 2021
1 parent f71488c commit 74e37bb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/route/test_static_route.py
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ def run_static_route_test(duthost, ptfadapter, ptfhost, tbinfo, prefix, nexthop_
# Config save and reload if specified
if config_reload_test:
duthost.shell('config save -y')
config_reload(duthost)
config_reload(duthost, wait=350)
generate_and_verify_traffic(duthost, ptfadapter, tbinfo, ip_dst, nexthop_devs, ipv6=ipv6)
check_route_redistribution(duthost, prefix, ipv6)

Expand Down

0 comments on commit 74e37bb

Please sign in to comment.