Skip to content

Commit

Permalink
increase lib/runtest.py timeout reserve to 50 seconds
Browse files Browse the repository at this point in the history
The original 10s was too little and long-running tests could hit
a time drift, causing a tmt error that appeared like

    custom results file not found in ...

because nothing provided the final '/' result (nothing called
report_and_exit()).

Signed-off-by: Jiri Jaburek <comps@nomail.dom>
  • Loading branch information
comps authored and mildas committed Dec 2, 2024
1 parent 97e1bc8 commit 8c29831
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions lib/runtest.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,10 @@ def _setup_timeout_handling():
metadata = util.TestMetadata()
duration = metadata.duration_seconds()

# leave 10 seconds for our alarm timeout code
duration -= 10
# leave 50 seconds for our alarm timeout code
# (the large-ish value is to allow for time drift between our timekeeping
# and tmt's timeout logic, for long-running tests)
duration -= 50

def _alarm_timed_out(signum, frame):
# sys.exit does run all cleanups (context manager, atexit, etc.),
Expand Down

0 comments on commit 8c29831

Please sign in to comment.