Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

parallel.test-worker-message-port-transfer-terminate is flaky on smartOS #56850

Open
anonrig opened this issue Jan 31, 2025 · 2 comments
Open
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. smartos Issues and PRs related to the SmartOS platform.

Comments

@anonrig
Copy link
Member

anonrig commented Jan 31, 2025

Test

parallel.test-worker-message-port-transfer-terminate

Platform

SmartOS

Console output

---
duration_ms: 362.04
exitcode: -6
severity: crashed
stack: |-
  terminate called after throwing an instance of 'std::bad_alloc'
    what():  std::bad_alloc
...

Build links

Additional information

cc @nodejs/platform-smartos

@anonrig anonrig added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label Jan 31, 2025
@github-actions github-actions bot added the smartos Issues and PRs related to the SmartOS platform. label Jan 31, 2025
@mcollina
Copy link
Member

mcollina commented Feb 3, 2025

This has been found flaky a few times before: https://github.com/search?q=repo%3Anodejs%2Freliability+test-worker-message-port-transfer-terminate&type=issues.

I have a rough issue this would have some timing issues that are unrelated to SmartOS

@jclulow
Copy link

jclulow commented Feb 3, 2025

My assumption, without having dug in too far just yet, is that std::bad_alloc is presumably just a failure to allocate memory. As part of nodejs/build#4011 we're looking at increasing the memory available to the CI workers, and perhaps most critically here we're going to substantially increase the available swap.

Unlike Linux systems, illumos systems do not overcommit memory; if we don't have a place to put the memory you allocate with malloc(3C), whether in physical memory or in a swap file, we fail the allocation. This is to avoid the need for dubious mechanisms like an OOM killer, etc.

My hope is that once we sort out nodejs/build#4011 that issues around memory allocation like this one will just go away!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. smartos Issues and PRs related to the SmartOS platform.
Projects
None yet
Development

No branches or pull requests

3 participants