GeezerVison

Stress test: https://ci.nodejs.org/view/Stress/job/node-stress-single-test/532/

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.89%. Comparing base (e1e312d) to head (4001157).
Report is 172 commits behind head on main.

@@            Coverage Diff             @@
##             main   #54840      +/-   ##
==========================================
- Coverage   87.90%   87.89%   -0.01%     
==========================================
  Files         651      651              
  Lines      183343   183343              
  Branches    35710    35712       +2     
==========================================
- Hits       161165   161155      -10     
+ Misses      15466    15461       -5     
- Partials     6712     6727      +15     

IIRC this made test more flaky on armv7 (32bit), do we still have these in stress CI?

@LiviaMedeiros .... can you expand on that a bit more? Maybe a link to prior attempts/discussion

Stress test covers:

can you expand on that a bit more? Maybe a link to prior attempts/discussion

b3d0804 (#52674) was an attempt to do the same. The source of flakiness was confirmed to be code using ErrorWithCyclicCause and the change indeed speeds up unwinding, however limiting stack size haven't solved the issue (basically, when the bug appears and test is not killed by timeout, it stayed in loop for 20+ minutes regardless of stack limit) and for whatever reason made it flake on armv7 even more frequently.
I don't see an option to start stress test CI on any 32bit arch though.

@LiviaMedeiros ... good fun.. ok. Did anyone try moving the test to the sequential suite rather than the parallel suite? Maybe there's just too much going on concurrently

The flake was reproducible locally (although not as frequent as in CI) using sequential for i in $(seq 9999); do echo $i; node test.js; done, but it's worth a try.

@LiviaMedeiros ... done! Moved the test over to sequential with the lower stack limit. Let's give this a try and see if it at least reduces how often it flakes.

heh, we kicked off CI at the same time lol.

Almost cancelled at the same time as well 😄
Stress CI just in case: https://ci.nodejs.org/job/node-stress-single-test/534/

CI: https://ci.nodejs.org/job/node-test-pull-request/62328/

Landed in 3783499