-
Notifications
You must be signed in to change notification settings - Fork 30.9k
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
AIX, FreeBSD: test-cluster-disconnect-handles intermittent failures #10609
Comments
Also FreeBSD. https://ci.nodejs.org/job/node-test-commit-freebsd/6249/nodes=freebsd10-64/console not ok 226 parallel/test-cluster-disconnect-handles
---
duration_ms: 60.78
severity: fail
stack: |-
timeout |
Seems like this comment in the test kinda sorta acknowledges that there is a race condition: // It can happen that the first continue was sent before the break
// event was received. If that's the case, send also a continue from
// here so the worker exits |
The test uses Not sure if the right thing to do is just get rid of the test or to rewrite it to use something else like /cc @nodejs/testing @nodejs/diagnostics @jkrems @s3ththompson @eugeneo |
Hopefully I didn't typo any of the things in the form, but stress tests for... AIX: https://ci.nodejs.org/job/node-stress-single-test/1097/nodes=aix61-ppc64/console FreeBSD: https://ci.nodejs.org/job/node-stress-single-test/1098/nodes=freebsd10-64/console |
Oh, maybe the failure could be due to a stuck debugger test on one of the hosts preventing usage of the TCP port. I know @thefourtheye has been trying to get the debugger tests working again, and that's involved running them in CI for the first time in a very long time. And some of them hung... |
/cc @bnoordhuis who wrote the test in the first place, especially on the question of whether it makes sense to port the test to something that doesn't use the to-be-removed debug protocol or whether it makes sense to simply delete the test. (Basically: Is the debug protocol part of what's actually being tested? Doesn't seem like it to me, but ¯\(ツ)/¯ ) |
Another failure here: https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix61-ppc64/3511/console |
I think this may also require cleanup after a failure as I saw it consistently fail a few times on the same machine and then there were processes hanging around on that machine. @bnoordhuis can you help out so that we can decide if we can just delete the test or if its needs to be fixed ? |
Another one: https://ci.nodejs.org/job/node-test-commit-aix/3513/nodes=aix61-ppc64/console not ok 128 parallel/test-cluster-disconnect-handles
---
duration_ms: 60.101
severity: fail
stack: |-
timeout |
Test was moved to |
I've seen a number of failures on this test over the last little while:
https://ci.nodejs.org/job/node-test-commit-aix/2942/nodes=aix61-ppc64/console
The text was updated successfully, but these errors were encountered: