Skip to content

test: deflake test-worker-stack-overflow-stack-size in the Node.js suite - #53075

Merged
MarshallOfSound merged 1 commit into
41-x-yfrom
trop/41-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306222545
Aug 21, 2026
Merged

MarshallOfSound merged 1 commit into
41-x-yfrom
trop/41-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306222545

Conversation

@trop

@trop trop Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Backport of #53062

See that PR for details.

Notes: none

Backports nodejs/node#64271. The test compares the recursion depth two
Workers reach before overflowing, but depth also depends on frame size,
which changes when the recursive function tiers up mid-recursion. On a
busy CI runner that lands non-deterministically and the depths end up
>10% apart. Pin the function to the interpreter so the depth is stable.

The fix is only in Node 26.6.0+, so it is carried as a patch until the
Node roll picks it up.

Co-authored-by: Sam Attard <[email protected]>
@trop
trop Bot requested a review from a team as a code owner August 21, 2026 09:57
@trop
trop Bot requested a review from MarshallOfSound August 21, 2026 09:57
@trop trop Bot added 41-x-y backport This is a backport PR semver/none labels Aug 21, 2026
@MarshallOfSound
MarshallOfSound merged commit ab5d598 into 41-x-y Aug 21, 2026
86 of 88 checks passed
@MarshallOfSound
MarshallOfSound deleted the trop/41-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306222545 branch August 21, 2026 11:01
@release-clerk

release-clerk Bot commented Aug 21, 2026

Copy link
Copy Markdown

No Release Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant