Skip to content

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

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

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

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 43-x-y backport This is a backport PR semver/none labels Aug 21, 2026
@MarshallOfSound
MarshallOfSound merged commit 3bbf42d into 43-x-y Aug 21, 2026
96 checks passed
@MarshallOfSound
MarshallOfSound deleted the trop/43-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306226122 branch August 21, 2026 11:02
@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

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant