Skip to content

Commit 6dcbc86

Browse files
vivekverma-aristamssonicbld
authored andcommitted
Fix qos/test_qos_sai.py teardown. (#11934)
Regression introduced by #10651 for dualtor. The config_reload in the fixture `dut_disable_ipv6` waits until all critical processes are up after issuing config reload command and it timeouts in case of dualtor because mux container doesn't come up. Mux container is disabled by another fixture in the same file `stopServices`. These two fixtures have no dependency on each other hence the execution can happen in any order, so if the teardown of `dut_disable_ipv6` happens before `stopServices` then this issue is seen.
1 parent 0208949 commit 6dcbc86

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/qos/qos_sai_base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1113,7 +1113,7 @@ def updateIptables(self, duthosts, get_src_dst_asic_and_duts, swapSyncd_on_selec
11131113

11141114
@pytest.fixture(scope='class')
11151115
def stopServices(
1116-
self, duthosts, get_src_dst_asic_and_duts,
1116+
self, duthosts, get_src_dst_asic_and_duts, dut_disable_ipv6,
11171117
swapSyncd_on_selected_duts, enable_container_autorestart, disable_container_autorestart, get_mux_status, # noqa F811
11181118
tbinfo, upper_tor_host, lower_tor_host, toggle_all_simulator_ports): # noqa F811
11191119
"""

0 commit comments

Comments
 (0)