Skipping reboot on Supervisor for Cisco 8000 distributed chassis#6991
Skipping reboot on Supervisor for Cisco 8000 distributed chassis#6991vperumal wants to merge 2 commits intosonic-net:masterfrom
Conversation
|
Can we add the test cases to be skipped here: https://github.com/sonic-net/sonic-mgmt/blob/master/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml. |
Hi @SuvarnaMeenakshi, the check for skip using test_mark_conditions happens at the start of the script even before dut is selected. It works fine in a single dut case but in case of multi-dut wherein it is randomly picked up within the testcase, test_mark_conditions does not work. Hence had to put it in each individual testcase. |
abdosi
left a comment
abdosi
left a comment
There was a problem hiding this comment.
please add close pid platforms also.
| """ | ||
| dut.command("config save -y") | ||
| if dut.is_supervisor_node and dut.facts['platform'] in ['x86_64-8800_rp_o-r0']: | ||
| pytest.skip("Skip reboot for Supervisor card in Cisco 8000 distributed chassis") |
There was a problem hiding this comment.
Remove this . this should not get call for supervisor.
|
we will not need now as reboot is working fine |
3 participants
Description of PR
Skipping reboot on Supervisor for Cisco 8000 distributed chassis.
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
Currently we are hitting an intermittent issue if Supervisor is rebooted during a test. To avoid tests to get errored out during nightly run, we are skipping reboot for supervisor node
How did you do it?
How did you verify/test it?
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation