-
Notifications
You must be signed in to change notification settings - Fork 30.4k
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 sequential/test-inspector-async-hook-setup-at-signal #16771
Comments
/cc @nodejs/v8-inspector @nodejs/platform-windows @nodejs/testing |
this line got me worried: |
Again today: not ok 501 sequential/test-inspector-async-hook-setup-at-signal
---
duration_ms: 0.597
severity: fail
stack: |-
[err] Waiting until a signal enables the inspector...
[err]
[err] started
[err]
[test] Connecting to a child Node process
[test] Testing /json/list
[err] Debugger listening on ws://127.0.0.1:9229/a7c1a76c-cdc2-4171-a585-bfd38c040cff
[err] For help see https://nodejs.org/en/docs/inspector
[err]
[err] Debugger attached.
[err] FATAL ERROR: v8::HandleScope::CreateHandle() Cannot create a handle without a HandleScope
[err]
{ Error: read ECONNRESET
at _errnoException (util.js:1031:13)
at TCP.onread (net.js:619:25) errno: 'ECONNRESET', code: 'ECONNRESET', syscall: 'read' }
1
... |
Another one. Might need to make this one a priority if this keeps up.... not ok 501 sequential/test-inspector-async-hook-setup-at-signal
---
duration_ms: 0.697
severity: fail
stack: |-
[err] Waiting until a signal enables the inspector...
[err]
[err] started
[err]
[test] Connecting to a child Node process
[test] Testing /json/list
[err] Debugger listening on ws://127.0.0.1:9229/ac5e85b4-c292-4a6e-b034-a55f59938e82
[err] For help see https://nodejs.org/en/docs/inspector
[err]
[err] Debugger attached.
[err] FATAL ERROR: v8::HandleScope::CreateHandle() Cannot create a handle without a HandleScope
[err]
{ Error: read ECONNRESET
at _errnoException (util.js:1031:13)
at TCP.onread (net.js:619:25) errno: 'ECONNRESET', code: 'ECONNRESET', syscall: 'read' }
1
... |
There is a timeout as well: https://ci.nodejs.org/job/node-test-commit-linux/15972/nodes=debian8-64/console
|
This hasn't been reported for a long time now and there is a PR open to unmark it as flaky. #22253 I'm going to close this, but as always, re-open if that's not the right thing to do. |
CI link
The text was updated successfully, but these errors were encountered: