diff --git a/bin/tests/system/isctest/mark.py b/bin/tests/system/isctest/mark.py index 805371701d..4d7fcce147 100644 --- a/bin/tests/system/isctest/mark.py +++ b/bin/tests/system/isctest/mark.py @@ -43,6 +43,10 @@ def is_dnsrps_available(): return True +def is_host_freebsd(*_): + return platform.system() == "FreeBSD" + + def is_host_freebsd_13(*_): return platform.system() == "FreeBSD" and platform.release().startswith("13") diff --git a/bin/tests/system/isctest/run.py b/bin/tests/system/isctest/run.py index 9b6ef31236..65fd5ec8df 100644 --- a/bin/tests/system/isctest/run.py +++ b/bin/tests/system/isctest/run.py @@ -142,13 +142,17 @@ def perl(script: str, args: Optional[List[str]] = None) -> None: def retry_with_timeout(func, timeout, delay=1, msg=None): start_time = time.monotonic() exc_msg = None + fname = f"{func.__module__}.{func.__qualname__}()" while time.monotonic() < start_time + timeout: exc_msg = None + isctest.log.debug(f"retry_with_timeout: {fname} called") try: if func(): + isctest.log.debug(f"retry_with_timeout: {fname} succeeded") return except AssertionError as exc: exc_msg = str(exc) + isctest.log.debug(f"retry_with_timeout: {fname} failed, sleep {delay}s") time.sleep(delay) if exc_msg is not None: isctest.log.error(exc_msg) @@ -156,7 +160,7 @@ def retry_with_timeout(func, timeout, delay=1, msg=None): if exc_msg is not None: msg = exc_msg else: - msg = f"{func.__module__}.{func.__qualname__} timed out after {timeout} s" + msg = f"{fname} timed out after {timeout} s" assert False, msg diff --git a/bin/tests/system/kasp/tests_kasp.py b/bin/tests/system/kasp/tests_kasp.py index 1a836a230d..89f2696e56 100644 --- a/bin/tests/system/kasp/tests_kasp.py +++ b/bin/tests/system/kasp/tests_kasp.py @@ -596,7 +596,7 @@ def cb_remove_keyfiles(params, ksks=None, zsks=None): }, id="secondary.kasp", marks=pytest.mark.flaky( - max_runs=2, rerun_filter=isctest.mark.is_host_freebsd_13 + max_runs=2, rerun_filter=isctest.mark.is_host_freebsd ), ), pytest.param(