Skip to content

Commit 82c3858

Browse files
vivekverma-aristamssonicbld
authored andcommitted
Fix fixture invocation order in qos/qos_sai_base.py (sonic-net#12152)
What is the motivation for this PR? qos/test_qos_sai.py fail with the following - failed on setup with "Failed: Not all critical processes are healthy" The fixture dut_disable_ipv6 disables ipv6 due to which the following critical processes don't come up after swapping syncd ( fixture swapSyncd_on_selected_duts ) - program:snmpd program:snmp-subagent And this issue only hits if swapSyncd_on_selected_duts is executed after dut_disable_ipv6 and since there is no dependency between these fixtures, they can execute in any order. How did you do it? This change introduces a dependency between dut_disable_ipv6 & swapSyncd_on_selected_duts such that former is always executed after latter. How did you verify/test it? Tested on Arista-7050CX3 platform with dualtor topology.
1 parent 0be1031 commit 82c3858

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

tests/qos/qos_sai_base.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1688,7 +1688,8 @@ def populateArpEntries(
16881688
return
16891689

16901690
@pytest.fixture(scope='class', autouse=True)
1691-
def dut_disable_ipv6(self, duthosts, get_src_dst_asic_and_duts, tbinfo, lower_tor_host): # noqa F811
1691+
def dut_disable_ipv6(self, duthosts, get_src_dst_asic_and_duts, tbinfo, lower_tor_host, # noqa F811
1692+
swapSyncd_on_selected_duts):
16921693
for duthost in get_src_dst_asic_and_duts['all_duts']:
16931694
docker0_ipv6_addr = \
16941695
duthost.shell("sudo ip -6 addr show dev docker0 | grep global" + " | awk '{print $2}'")[

0 commit comments

Comments
 (0)