-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
investigate flaky test-domain-error-types #38063
Comments
This one has been happening pretty relentlessly. It feels superfluous to add more examples, but here's one anyway.
|
(Seems to only happen in debug builds but happens there a lot.) |
The first time this appears to happen in v8 canary build 3 days ago: https://ci.nodejs.org/job/node-test-commit-node-v8/765/ That's a sub-task of https://ci.nodejs.org/job/node-update-v8-canary/1714/. Weirdly, the main task is green even though the subtask is red. @nodejs/v8 |
The first time this shows up in an actual pull request CI seems to be https://ci.nodejs.org/job/node-test-commit-linux-containered/26102/ which is a CI run for #37991 which is surely unrelated. So...something in V8 that slipped in at some point? |
(Or maybe some other change messed with garbage collection or something in a way that surfaces a bug that was masked previously?) |
Now they're all green. Starting to seem like I was wasting my time bisecting because the issue was a CI issue and not an issue with the test? |
Optimistically closing. |
Argh, just did a resume build on node-daily-master and it showed up again. |
Failed again today: https://ci.nodejs.org/job/node-test-commit-linux-containered/27102/ |
This now has been consistently failing https://ci.nodejs.org/job/node-test-commit-arm-debug/ (now on the equinix-ubuntu1804_sharedlibs_container-arm64 containers) |
There is no recent trace of failure for this test. Fixes: nodejs#38063
There is no recent trace of failure for this test. Fixes: #38063 PR-URL: #51717 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
There is no recent trace of failure for this test. Fixes: nodejs#38063 PR-URL: nodejs#51717 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
There is no recent trace of failure for this test. Fixes: #38063 PR-URL: #51717 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
There is no recent trace of failure for this test. Fixes: #38063 PR-URL: #51717 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
The text was updated successfully, but these errors were encountered: