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 crashing known_issues/test-repl-require-context on debian8-docker-armv7 in CI #24643

Closed
Trott opened this issue Nov 25, 2018 · 6 comments
Labels
arm Issues and PRs related to the ARM platform. build Issues and PRs related to build files or the CI. flaky-test Issues and PRs related to the tests with unstable failures on the CI.

Comments

@Trott
Copy link
Member

Trott commented Nov 25, 2018

Test: test file - known_issues/test-repl-require-context
Job: https://ci.nodejs.org/job/node-test-commit-arm/nodes=debian8-docker-armv7/20295/
Worker: test-scaleway-ubuntu1604-armv7l-3
Known-issue Refs: #7788

19:10:45 not ok 2320 known_issues/test-repl-require-context
19:10:45   ---
19:10:45   duration_ms: 0.929
19:10:45   severity: crashed
19:10:45   exitcode: -11
19:10:45   stack: |-
19:10:45     assert.js:86
19:10:45       throw new AssertionError(obj);
19:10:45       ^
19:10:45     
19:10:45     AssertionError [ERR_ASSERTION]: Expected values to be strictly deep-equal:
19:10:45     + actual - expected
19:10:45     
19:10:45       [
19:10:45         'undefined',
19:10:45         'true',
19:10:45     +   'false',
19:10:45     -   'true',
19:10:45         ''
19:10:45       ]
19:10:45         at REPLServer.r.on.common.mustCall (/home/iojs/build/workspace/node-test-commit-arm/nodes/debian8-docker-armv7/test/known_issues/test-repl-require-context.js:24:10)
19:10:45         at REPLServer.<anonymous> (/home/iojs/build/workspace/node-test-commit-arm/nodes/debian8-docker-armv7/test/common/index.js:346:15)
19:10:45         at REPLServer.emit (events.js:189:13)
19:10:45         at REPLServer.EventEmitter.emit (domain.js:434:20)
19:10:45         at REPLServer.emitExit (repl.js:559:10)
19:10:45         at REPLServer.emit (events.js:194:15)
19:10:45         at REPLServer.EventEmitter.emit (domain.js:434:20)
19:10:45         at REPLServer.Interface.close (readline.js:422:8)
19:10:45         at process.nextTick (repl.js:795:31)
19:10:45         at internalTickCallback (internal/process/next_tick.js:70:11)
19:10:45   ...

[refack: edited info and added link to tests file]

@Trott Trott added build Issues and PRs related to build files or the CI. arm Issues and PRs related to the ARM platform. flaky-test Issues and PRs related to the tests with unstable failures on the CI. labels Nov 25, 2018
@Trott
Copy link
Member Author

Trott commented Nov 25, 2018

https://ci.nodejs.org/job/node-test-commit-arm/nodes=debian8-docker-armv7/20306/console

Host is test-scaleway-ubuntu1604-armv7l-3

00:33:21 not ok 2323 known_issues/test-repl-require-context
00:33:21   ---
00:33:21   duration_ms: 0.827
00:33:21   severity: crashed
00:33:21   exitcode: -11
00:33:21   stack: |-
00:33:21     assert.js:86
00:33:21       throw new AssertionError(obj);
00:33:21       ^
00:33:21     
00:33:21     AssertionError [ERR_ASSERTION]: Expected values to be strictly deep-equal:
00:33:21     + actual - expected
00:33:21     
00:33:21       [
00:33:21         'undefined',
00:33:21         'true',
00:33:21     +   'false',
00:33:21     -   'true',
00:33:21         ''
00:33:21       ]
00:33:21         at REPLServer.r.on.common.mustCall (/home/iojs/build/workspace/node-test-commit-arm/nodes/debian8-docker-armv7/test/known_issues/test-repl-require-context.js:24:10)
00:33:21         at REPLServer.<anonymous> (/home/iojs/build/workspace/node-test-commit-arm/nodes/debian8-docker-armv7/test/common/index.js:346:15)
00:33:21         at REPLServer.emit (events.js:189:13)
00:33:21         at REPLServer.EventEmitter.emit (domain.js:434:20)
00:33:21         at REPLServer.emitExit (repl.js:559:10)
00:33:21         at REPLServer.emit (events.js:194:15)
00:33:21         at REPLServer.EventEmitter.emit (domain.js:434:20)
00:33:21         at REPLServer.Interface.close (readline.js:422:8)
00:33:21         at process.nextTick (repl.js:795:31)
00:33:21         at internalTickCallback (internal/process/next_tick.js:70:11)
00:33:21   ...

@Trott
Copy link
Member Author

Trott commented Nov 25, 2018

@nodejs/platform-arm @nodejs/docker @nodejs/build

@Trott
Copy link
Member Author

Trott commented Nov 25, 2018

Oh, wait, is it not really crashing and just reporting "severity: crashed"? I wouldn't expect an AssertionError on a crash, obviously....

@Trott
Copy link
Member Author

Trott commented Nov 25, 2018

(Output can be tricky to read perhaps because it's a known_issues test so the AssertionError may be expected--required even.)

@refack
Copy link
Contributor

refack commented Nov 25, 2018

Oh, wait, is it not really crashing and just reporting "severity: crashed"?

AFAICT it is supposed to emit the assertion error.
It is not supposed to crash with code -11.

@Trott
Copy link
Member Author

Trott commented Feb 18, 2019

Seems to not be an issue anymore.

@Trott Trott closed this as completed Feb 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arm Issues and PRs related to the ARM platform. build Issues and PRs related to build files or the CI. flaky-test Issues and PRs related to the tests with unstable failures on the CI.
Projects
None yet
Development

No branches or pull requests

2 participants