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 test-worker-messageport-transfer-terminate #25134

Closed
joyeecheung opened this issue Dec 19, 2018 · 3 comments
Closed

Investigate test-worker-messageport-transfer-terminate #25134

joyeecheung opened this issue Dec 19, 2018 · 3 comments
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. worker Issues and PRs related to Worker support.

Comments

@joyeecheung
Copy link
Member

joyeecheung commented Dec 19, 2018

Reason parallel/test-worker-messageport-transfer-terminate
Type JS_TEST_FAILURE
Failed PR 2 (#25125, #25088)
Appeared test-linuxonecc-rhel72-s390x-1, test-rackspace-ubuntu1604-x64-2
First CI https://ci.nodejs.org/job/node-test-pull-request/19675/
Last CI https://ci.nodejs.org/job/node-test-pull-request/19679/

Example

not ok 2085 parallel/test-worker-messageport-transfer-terminate
  ---
  duration_ms: 0.829
  severity: crashed
  exitcode: -6
  stack: |-
    FATAL ERROR: v8::ToLocalChecked Empty MaybeLocal.
     1: 0x15c1bd4 node::Abort() [out/Release/node]
     2: 0x15c27e6 node::OnFatalError(char const*, char const*) [out/Release/node]
     3: 0x17cdc58 v8::Utils::ReportApiFailure(char const*, char const*) [out/Release/node]
     4: 0x1691e82  [out/Release/node]
     5: 0x159d500 node::binding::GetInternalBinding(v8::FunctionCallbackInfo<v8::Value> const&) [out/Release/node]
     6: 0x1876648  [out/Release/node]
     7: 0x18771c6 v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*) [out/Release/node]
     8: 0x278f75c  [out/Release/node]
  ...

@joyeecheung joyeecheung added flaky-test Issues and PRs related to the tests with unstable failures on the CI. worker Issues and PRs related to Worker support. labels Dec 19, 2018
@joyeecheung
Copy link
Member Author

I walked through the recent 100 CI runs and this failure seems to only happen in test-worker-messageport-transfer-terminate, so I suspect the internalBinding causing this should be either worker or messaging.

cc @nodejs/workers @addaleax

@joyeecheung
Copy link
Member Author

joyeecheung commented Dec 19, 2018

The first failure in CI is rebasing onto #25076 (which added this test)

@addaleax
Copy link
Member

I can reproduce this locally.

The binding in question should be uv, based on the disassembly of my failed build, so I would be reasonably certain this is being addressed by #25079 – I’ll land it, add a Fixes: tag and we can re-open if necessary.

MylesBorins pushed a commit that referenced this issue Dec 25, 2018
This can fail when terminating a Worker that loads
the `uv` binding at the same time.

Refs: #25061 (comment)
Fixes: #25134
PR-URL: #25079
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
refack pushed a commit to refack/node that referenced this issue Jan 14, 2019
This can fail when terminating a Worker that loads
the `uv` binding at the same time.

Refs: nodejs#25061 (comment)
Fixes: nodejs#25134
PR-URL: nodejs#25079
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Trott pushed a commit to Trott/io.js that referenced this issue Jul 26, 2019
This can fail when terminating a Worker that loads
the `uv` binding at the same time.

Refs: nodejs#25061 (comment)
Fixes: nodejs#25134
PR-URL: nodejs#25079
Backport-PR-URL: nodejs#28832
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
BethGriggs pushed a commit that referenced this issue Jul 30, 2019
This can fail when terminating a Worker that loads
the `uv` binding at the same time.

Refs: #25061 (comment)
Fixes: #25134
PR-URL: #25079
Backport-PR-URL: #28832
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. worker Issues and PRs related to Worker support.
Projects
None yet
Development

No branches or pull requests

2 participants