summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Breunig <christian@breunig.cc>2024-01-07 21:59:37 +0100
committerGitHub <noreply@github.com>2024-01-07 21:59:37 +0100
commit44c190dd44f82dad5678a5b4220bb9551b570e2d (patch)
treebe21d114480138a51719d3d33e778814838fe826
parent3ee63a4f4cbc17da1e0e8bce5ea36af0c4a0928b (diff)
parent041db49533d57cabfccd319492b85ee0bafdd40c (diff)
downloadvyos-1x-44c190dd44f82dad5678a5b4220bb9551b570e2d.tar.gz
vyos-1x-44c190dd44f82dad5678a5b4220bb9551b570e2d.zip
Merge pull request #2769 from c-po/T5195-penalty
smoketest: T5195: fix BasicInterfaceTest tearDown() timeout penalty
-rw-r--r--python/vyos/utils/process.py2
-rw-r--r--smoketest/scripts/cli/base_interfaces_test.py4
2 files changed, 3 insertions, 3 deletions
diff --git a/python/vyos/utils/process.py b/python/vyos/utils/process.py
index cd58b4be2..bd0644bc0 100644
--- a/python/vyos/utils/process.py
+++ b/python/vyos/utils/process.py
@@ -204,7 +204,7 @@ def process_running(pid_file):
pid = f.read().strip()
return pid_exists(int(pid))
-def process_named_running(name, cmdline: str=None, timeout=0):
+def process_named_running(name: str, cmdline: str=None, timeout: int=0):
""" Checks if process with given name is running and returns its PID.
If Process is not running, return None
"""
diff --git a/smoketest/scripts/cli/base_interfaces_test.py b/smoketest/scripts/cli/base_interfaces_test.py
index 7219fe622..a40b762a8 100644
--- a/smoketest/scripts/cli/base_interfaces_test.py
+++ b/smoketest/scripts/cli/base_interfaces_test.py
@@ -127,9 +127,9 @@ class BasicInterfaceTest:
# by also checking the cmd arguments passed to the daemon
if self._interfaces:
for tmp in self._interfaces:
- self.assertFalse(process_named_running(daemon, tmp, timeout=10))
+ self.assertFalse(process_named_running(daemon, tmp))
else:
- self.assertFalse(process_named_running(daemon, timeout=10))
+ self.assertFalse(process_named_running(daemon))
def test_dhcp_disable_interface(self):
if not self._test_dhcp: