From cbbf95ee55b37181405d1fb635edafc384a64f45 Mon Sep 17 00:00:00 2001 From: Fabian Boehm Date: Mon, 13 Jan 2025 15:17:16 +0100 Subject: [PATCH] tests: Increase another timeout Failed on Cirrus Alpine. The only explanation I can come up with here is that this took over 100ms to start `true | sleep 6`. The alternative is that it started it and then did not regain control in 6 seconds to kill that sleep. Part of #11036 --- tests/pexpects/job_summary.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/pexpects/job_summary.py b/tests/pexpects/job_summary.py index fa3d0d84e..00337003b 100644 --- a/tests/pexpects/job_summary.py +++ b/tests/pexpects/job_summary.py @@ -55,8 +55,8 @@ expect_prompt() # fish_job_summary is called when foreground job is signalled. # cmd_line contains the entire pipeline. proc_id and proc_name are set in a pipeline. sendline("true | sleep 6") -sleep(0.100) +sleep(0.200) # Beware: Mac pkill requires that the -P argument come before the process name, # else the -P argument is ignored. call(["pkill", "-KILL", "-P", str(sp.spawn.pid), "sleep"]) -expect_re("[0-9]+:1:true|sleep 6:SIGKILL:Forced quit:[0-9]+:sleep", timeout=20) +expect_re("[0-9]+:1:true|sleep 6:SIGKILL:Forced quit:[0-9]+:sleep", timeout=10)