Skip to content

Commit

Permalink
Modify interface recovery to support multi-asic (#5087)
Browse files Browse the repository at this point in the history
What is the motivation for this PR?
Modify config interface command to take namespace parameter.
When running any test case with Sanity tests, if the testbed is in bad state and recovery operation is done, then it results in the error missing  -n namespace option.

How did you do it?
Modified interface recovery to startup interface in a specific asic namespace.

How did you verify/test it?
Bring up single and multi-asic vs testbeds.
Put the DUT in bad state by shutting any of the up interface.
Run the basic bgp facts test on single and mult-asic VS testbed, where one interface is down.
  • Loading branch information
SuvarnaMeenakshi authored Feb 16, 2022
1 parent 4afbb8c commit 79a8403
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/common/plugins/sanity_check/recover.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,8 @@ def __recover_interfaces(dut, fanouthosts, result, wait_time):
fanout, fanout_port = fanout_switch_port_lookup(fanouthosts, dut.hostname, port)
if fanout and fanout_port:
fanout.no_shutdown(fanout_port)
dut.no_shutdown(port)
asic = dut.get_port_asic_instance(port)
dut.asic_instance(asic.asic_index).startup_interface(port)
wait(wait_time, msg="Wait {} seconds for interface(s) to restore.".format(wait_time))
return action

Expand Down

0 comments on commit 79a8403

Please sign in to comment.