Skip to content

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

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

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

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 44-x-y backport This is a backport PR semver/none labels Aug 21, 2026
@MarshallOfSound
MarshallOfSound merged commit 8fe296d into 44-x-y Aug 21, 2026
90 of 91 checks passed
@release-clerk

release-clerk Bot commented Aug 21, 2026

Copy link
Copy Markdown

No Release Notes

@MarshallOfSound
MarshallOfSound deleted the trop/44-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306225324 branch August 21, 2026 11:01
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