From 2207931be40b5820054e0f1b0e4b8dea4f43481f Mon Sep 17 00:00:00 2001 From: Chris <156943338+ccroy-arista@users.noreply.github.com> Date: Tue, 16 Apr 2024 23:38:02 -0700 Subject: [PATCH] Replace immediate daemon running check in PSU status tests with a wait loop to provide some flexibility in the amount of time it takes for the daemons to go from STARTING state to RUNNING. (#12047) --- tests/platform_tests/cli/test_show_platform.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/platform_tests/cli/test_show_platform.py b/tests/platform_tests/cli/test_show_platform.py index b0f8c37f70c..5cfff929bf1 100644 --- a/tests/platform_tests/cli/test_show_platform.py +++ b/tests/platform_tests/cli/test_show_platform.py @@ -231,7 +231,7 @@ def test_show_platform_psustatus(duthosts, enum_supervisor_dut_hostname): duthost = duthosts[enum_supervisor_dut_hostname] logging.info("Check pmon daemon status on dut '{}'".format(duthost.hostname)) pytest_assert( - check_pmon_daemon_status(duthost), + wait_until(60, 5, 0, check_pmon_daemon_status, duthost), "Not all pmon daemons running on '{}'".format(duthost.hostname) ) cmd = " ".join([CMD_SHOW_PLATFORM, "psustatus"]) @@ -264,7 +264,9 @@ def test_show_platform_psustatus_json(duthosts, enum_supervisor_dut_hostname): pytest.skip("JSON output not available in this version") logging.info("Check pmon daemon status") - pytest_assert(check_pmon_daemon_status(duthost), "Not all pmon daemons running.") + pytest_assert( + wait_until(60, 5, 0, check_pmon_daemon_status, duthost), + "Not all pmon daemons running.") cmd = " ".join([CMD_SHOW_PLATFORM, "psustatus", "--json"])