Fuzzer build “Timed out!” #21639

1. 
   ryanofsky commented at 7:54 pm on April 8, 2021: member

   Fuzzer build [no depends, sanitizers: fuzzer,address,undefined,integer] [focal] seems to timing out on master:

   https://github.com/bitcoin/bitcoin/runs/2294417904

   Also one of my PRs:

   https://cirrus-ci.com/task/5980404822835200?logs=ci#L4543

   From the output it seems like it is succeeding it takes two hours and the output ends in:

   0Done 12648 runs in 255 second(s)
   1Timed out!
   

   There was a comment a few days ago about a fuzzer time out but I’m not sure it’s related #21609 (comment)

2. ryanofsky added the label Bug on Apr 8, 2021
3. 
   fanquake commented at 0:36 am on April 9, 2021: member

   This should be fixed by #21445.
4. fanquake added the label Tests on Apr 9, 2021
5. MarcoFalke closed this on Apr 9, 2021

   ---

6. 
   ryanofsky commented at 12:05 pm on April 9, 2021: member

   Thanks!

   I did see more recent “Timed out!” failures after this was merged. I wonder if this is expected or I need to rebase my PRs?

   Apr 8, 2021, 9:53 PM EDT - #21445 Merged Apr 8, 2021, 10:06 PM EDT - Restarted job which timed out https://cirrus-ci.com/task/6394351371681792 Apr 8, 2021, 10:11 PM EDT - Restarted job which timed out https://cirrus-ci.com/task/6058699123851264

7. 
   MarcoFalke commented at 2:16 pm on April 9, 2021: member

   The fix changes the task configuration, so re-running the same task won’t fix it. Any way that changes the commit id of the pull request will update the task configuration.
8. 
   ryanofsky commented at 3:21 pm on April 9, 2021: member

   Thanks again, will retry. I did notice updated .cirrus.yml files merged in these runs https://cirrus-ci.com/task/6394351371681792?logs=merge_base#L389, but it makes sense that the merge might happen too late to affect task configuration.
9. 
   ryanofsky commented at 3:23 pm on April 9, 2021: member

   Actually, I’m curious why changing the commit id without rebasing will update the task configuration? Won’t the task configuration still be pulled from refs/pull/<pr>/head not master?
10. 
    MarcoFalke commented at 4:47 pm on April 9, 2021: member

    For pull requests, Cirrus will merge with the target brach. See https://cirrus-ci.com/settings/repository/6264162542157824 which says “Merge for PRs”.
11. DrahtBot locked this on Aug 18, 2022

Contributors
ryanofsky fanquake MarcoFalke

Labels
Bug Tests