Skip to content

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

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

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

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 <sattard@anthropic.com>
@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 42-x-y backport This is a backport PR semver/none labels Aug 21, 2026
@MarshallOfSound
MarshallOfSound merged commit 01494a8 into 42-x-y Aug 21, 2026
90 of 91 checks passed
@MarshallOfSound
MarshallOfSound deleted the trop/42-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306223862 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