<div dir="auto">no hurry... it's taken us this long to spot any flake, so it'll probably be as long again before we see it again even if we do nothing!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 12, 2024, 09:54 Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Catching up. (I let stuff pile up preparing for the release and then took a<br>
couple days off, and now I'm at texas linuxfest doing sleep deprived talk prep<br>
for tomorrow...)<br>
<br>
On 4/8/24 15:28, enh via Toybox wrote:<br>
> A (presumably overloaded) CI server saw the `exit 0` test time out.<br>
> Given that several of these tests should just fail immediately,<br>
> having a huge timeout isn't even a bad thing --- if we had a bug<br>
> that caused us to report the correct status, but not until the<br>
> timeout had _also_ expired, this would make that failure glaringly<br>
> obvious.<br>
> <br>
> Aren't the other tests with 0.1s timeouts potentially flaky? Yes,<br>
> obviously, but I'll worry about those if/when we see them in real<br>
> life? (Because increasing those timeouts _would_ increase overall<br>
> test time.)<br>
<br>
Yes it should never happen, but 11 minutes seems like a footgun.<br>
<br>
I bumped it up to 1 second (10 times as long as before). If you see it again I<br>
can bump it to 5 seconds, but much beyond 1 second and the "timeout -v .1 sleep<br>
3" test later on gets flaky, as does:<br>
<br>
toyonly testcmd "-i" \<br>
  "-i 1 sh -c 'for i in .25 .50 2; do sleep \$i; echo hello; done'" \<br>
  "hello\nhello\n" "" ""<br>
<br>
Rob<br>
</blockquote></div>