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

Investigate flaky sequential/test-inspector-async-hook-setup-at-signal #16771

Closed
mscdex opened this issue Nov 5, 2017 · 6 comments
Closed

Investigate flaky sequential/test-inspector-async-hook-setup-at-signal #16771

mscdex opened this issue Nov 5, 2017 · 6 comments
Labels
async_hooks Issues and PRs related to the async hooks subsystem. flaky-test Issues and PRs related to the tests with unstable failures on the CI. inspector Issues and PRs related to the V8 inspector protocol test Issues and PRs related to the tests. windows Issues and PRs related to the Windows platform.

Comments

@mscdex
Copy link
Contributor

mscdex commented Nov 5, 2017

  • Version: master
  • Platform: windows
  • Subsystem: inspector, async_hooks

CI link

not ok 502 sequential/test-inspector-async-hook-setup-at-signal
  ---
  duration_ms: 0.693
  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/69903fae-3d0f-4fe0-baed-5c05272ba207
    [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
@mscdex mscdex added async_hooks Issues and PRs related to the async hooks subsystem. flaky-test Issues and PRs related to the tests with unstable failures on the CI. inspector Issues and PRs related to the V8 inspector protocol test Issues and PRs related to the tests. windows Issues and PRs related to the Windows platform. labels Nov 5, 2017
@mscdex
Copy link
Contributor Author

mscdex commented Nov 5, 2017

/cc @nodejs/v8-inspector @nodejs/platform-windows @nodejs/testing

@refack
Copy link
Contributor

refack commented Nov 5, 2017

this line got me worried: [err] FATAL ERROR: v8::HandleScope::CreateHandle() Cannot create a handle without a HandleScope, since AFAIK HandleScope should be RAII in a deterministic way ?

@Trott
Copy link
Member

Trott commented Nov 6, 2017

Again today:

https://ci.nodejs.org/job/node-test-binary-windows/12556/COMPILED_BY=vs2015,RUNNER=win2008r2-vs2017,RUN_SUBSET=3/console

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
  ...

@Trott
Copy link
Member

Trott commented Nov 6, 2017

Another one. Might need to make this one a priority if this keeps up....

https://ci.nodejs.org/job/node-test-binary-windows/12558/COMPILED_BY=vs2015,RUNNER=win2008r2-vs2017,RUN_SUBSET=3/console

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
  ...

@BridgeAR
Copy link
Member

BridgeAR commented Feb 1, 2018

There is a timeout as well:

https://ci.nodejs.org/job/node-test-commit-linux/15972/nodes=debian8-64/console

  duration_ms: 120.35
  severity: fail
  stack: |-
    timeout

@Trott
Copy link
Member

Trott commented Aug 10, 2018

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.

@Trott Trott closed this as completed Aug 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
async_hooks Issues and PRs related to the async hooks subsystem. flaky-test Issues and PRs related to the tests with unstable failures on the CI. inspector Issues and PRs related to the V8 inspector protocol test Issues and PRs related to the tests. windows Issues and PRs related to the Windows platform.
Projects
None yet
Development

No branches or pull requests

4 participants