From 702a60a8de28963146cbe77f3247d89a326cff3a Mon Sep 17 00:00:00 2001 From: Christian Breunig Date: Mon, 23 Dec 2024 13:35:00 +0100 Subject: smoketest: T6746: wait after commit() until frr-reload is no longer running This extends commit 90e9aa9df ("frr: T6746: add guard time after cli_commit() and before getFRRconfig()"). --- smoketest/scripts/cli/base_vyostest_shim.py | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'smoketest') diff --git a/smoketest/scripts/cli/base_vyostest_shim.py b/smoketest/scripts/cli/base_vyostest_shim.py index d95071d1a..9cf6a653a 100644 --- a/smoketest/scripts/cli/base_vyostest_shim.py +++ b/smoketest/scripts/cli/base_vyostest_shim.py @@ -27,6 +27,7 @@ from vyos import ConfigError from vyos.defaults import commit_lock from vyos.utils.process import cmd from vyos.utils.process import run +from vyos.utils.process import process_named_running save_config = '/tmp/vyos-smoketest-save' @@ -88,6 +89,9 @@ class VyOSUnitTestSHIM: # during a commit there is a process opening commit_lock, and run() returns 0 while run(f'sudo lsof -nP {commit_lock}') == 0: sleep(0.250) + # wait for FRR reload to be complete + while process_named_running('frr-reload.py'): + sleep(0.250) # reset getFRRconfig() guard timer self.commit_guard = time() -- cgit v1.2.3 From e1d34e5624ddaac4b8d3fcf69e2387461b4177b1 Mon Sep 17 00:00:00 2001 From: Christian Breunig Date: Mon, 23 Dec 2024 13:36:00 +0100 Subject: smoketest: T5791: add guard timeout when checking for ddclient process This extends commit c2d84b03d ("T5791: Modify ddclient smoketest for reliability"). --- smoketest/scripts/cli/test_service_dns_dynamic.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'smoketest') diff --git a/smoketest/scripts/cli/test_service_dns_dynamic.py b/smoketest/scripts/cli/test_service_dns_dynamic.py index 3ce459e44..28380f6d4 100755 --- a/smoketest/scripts/cli/test_service_dns_dynamic.py +++ b/smoketest/scripts/cli/test_service_dns_dynamic.py @@ -44,7 +44,7 @@ class TestServiceDDNS(VyOSUnitTestSHIM.TestCase): def tearDown(self): # Check for running process - self.assertTrue(process_named_running(DDCLIENT_PNAME)) + self.assertTrue(process_named_running(DDCLIENT_PNAME, timeout=5)) # Delete DDNS configuration self.cli_delete(base_path) -- cgit v1.2.3