From 610bc01eacf22fb28f186567ebf2bcfd49924799 Mon Sep 17 00:00:00 2001 From: xwjiang-ms Date: Thu, 9 Feb 2023 08:25:47 +0000 Subject: [PATCH 1/2] Add wait time to verify mux status What is the motivation for this PR? After config reload, mux simulator needs some time to switch to correct status How did you do it? Add wait time in sanity check to verify mux status --- tests/common/plugins/sanity_check/checks.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/common/plugins/sanity_check/checks.py b/tests/common/plugins/sanity_check/checks.py index 340dd4554a1..55257faa399 100644 --- a/tests/common/plugins/sanity_check/checks.py +++ b/tests/common/plugins/sanity_check/checks.py @@ -539,7 +539,7 @@ def _verify_show_mux_status(): duts_parsed_mux_status = {} err_msg_from_mux_status = [] if (has_active_active_ports and not wait_until(30, 5, 0, _verify_show_mux_status)) \ - or (not _verify_show_mux_status()): + or (not wait_until(30, 5, 0, _verify_show_mux_status)): if err_msg_from_mux_status: err_msg = err_msg_from_mux_status[-1] else: From 45ba1bfeec29dc1ab22193b2bcf7df6a0e4edf05 Mon Sep 17 00:00:00 2001 From: xiaweijiang Date: Wed, 1 Mar 2023 13:16:51 +0000 Subject: [PATCH 2/2] Check mux status later --- tests/common/plugins/sanity_check/checks.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/common/plugins/sanity_check/checks.py b/tests/common/plugins/sanity_check/checks.py index 55257faa399..239ec942cee 100644 --- a/tests/common/plugins/sanity_check/checks.py +++ b/tests/common/plugins/sanity_check/checks.py @@ -25,8 +25,8 @@ 'check_bgp', 'check_dbmemory', 'check_monit', - 'check_mux_simulator', - 'check_secureboot'] + 'check_secureboot', + 'check_mux_simulator',] __all__ = CHECK_ITEMS