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

Flaky heapdump pummel test #10

Open
trevnorris opened this issue Oct 27, 2023 · 0 comments
Open

Flaky heapdump pummel test #10

trevnorris opened this issue Oct 27, 2023 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@trevnorris
Copy link
Member

trevnorris commented Oct 27, 2023

Right now test/pummel/test-heapdump-worker.js is being skipped on asan builds because it sometimes fails showing indrect leaks. To demonstrate this the test will need to be edited to remove the process.config.variables.asan check. Then do the following:

./configure --enable-asan
make -j16
for i in $(seq 1 16); do ./out/Release/nsolid --expose-internals test/pummel/test-heapdump-worker.js & done

I assume this is a timing issue because it's harder to get the same failure if using both --debug and --enable-asan. It also only shows up if validating the snapshot then running worker.terminate(). Removing the validateSnapshotNodes() call and replacing it with a busy loop will also not trigger the issue.

@trevnorris trevnorris added the bug Something isn't working label Oct 27, 2023
@trevnorris trevnorris self-assigned this Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant