Skip to content

Commit c059460

Browse files
cyw233venu-nexthop
authored andcommitted
fix: correct dut variable name in startup tsa pretest (sonic-net#18699)
Description of PR Fix the dut variable name in test_disable_startup_tsa_tsb_service() in test_pretest.py added in sonic-net#17247 Summary: Fixes # (issue) Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 202405 202411 202505 Approach What is the motivation for this PR? When using multithreading parallel run, we cannot refer to the duthost variable, we need to use dut instead.
1 parent 5b9b1aa commit c059460

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tests/test_pretest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -461,7 +461,7 @@ def disable_startup_tsa_tsb(dut):
461461
dut.shell("sudo mv {} {}".format(startup_tsa_tsb_file_path, backup_tsa_tsb_file_path))
462462
output = dut.shell("TSB", module_ignore_errors=True)
463463
pytest_assert(not output['rc'], "Failed TSB")
464-
duthost.shell("sudo config save -y")
464+
dut.shell("sudo config save -y")
465465
else:
466466
logger.info("{} file does not exist in the specified path on dut {}".
467467
format(startup_tsa_tsb_file_path, dut.hostname))

0 commit comments

Comments
 (0)