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

test: flaky - parallel/test-pipe-stream #14128

Closed
refack opened this issue Jul 7, 2017 · 5 comments · Fixed by #14168
Closed

test: flaky - parallel/test-pipe-stream #14128

refack opened this issue Jul 7, 2017 · 5 comments · Fixed by #14168
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. stream Issues and PRs related to the stream subsystem. test Issues and PRs related to the tests.

Comments

@refack
Copy link
Contributor

refack commented Jul 7, 2017

  • Version: master
  • Platform: freeBSD
  • Subsystem: test,stream
not ok 912 parallel/test-pipe-stream
  ---
  duration_ms: 3.136
  severity: fail
  stack: |-
    events.js:182
          throw er; // Unhandled 'error' event
          ^
    
    Error: listen EADDRINUSE /usr/home/iojs/node-tmp/tmp.1/test.sock
        at Object.exports._errnoException (util.js:1023:11)
        at exports._exceptionWithHostPort (util.js:1046:20)
        at Server.setupListenHandle [as _listen2] (net.js:1302:19)
        at listenInCluster (net.js:1367:12)
        at Server.listen (net.js:1478:5)
        at test (/usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/test/parallel/test-pipe-stream.js:56:10)
        at /usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/test/parallel/test-pipe-stream.js:66:3
        at Server.<anonymous> (/usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/test/parallel/test-pipe-stream.js:61:11)
        at emitNone (events.js:105:13)
        at Server.emit (events.js:207:7)
  ...

https://ci.nodejs.org/job/node-test-commit-freebsd/10201/nodes=freebsd10-64/

@refack refack added flaky-test Issues and PRs related to the tests with unstable failures on the CI. stream Issues and PRs related to the stream subsystem. test Issues and PRs related to the tests. labels Jul 7, 2017
@refack
Copy link
Contributor Author

refack commented Jul 7, 2017

/cc @nodejs/platform-freebsd @nodejs/streams

@mcollina
Copy link
Member

The problem starts because we are not rotating/changing common.PIPE and https://github.com/nodejs/node/blob/master/test/common/index.js#L259-L266.
If there is another process left there, or if that is not cleaned up correctly, this will fail.

This should be done by https://github.com/nodejs/node/blob/master/test/common/index.js#L156-L159 called from https://github.com/nodejs/node/blob/master/test/parallel/test-pipe-stream.js#L6, but it seems not happening on freebsd.

@refack
Copy link
Contributor Author

refack commented Jul 11, 2017

The problem starts because we are not rotating/changing common.PIPE

@mcollina tl;dr you think best solution is for common to rotate common.PIPE? If yes, I'll open a PR.

@mcollina
Copy link
Member

Yes, I think so. Or to do one based on the process PID.

@refack
Copy link
Contributor Author

refack commented Jul 11, 2017

PR #14168 (Untested) CI ✔️

refack added a commit to refack/node that referenced this issue Jul 14, 2017
* includes a tiny bit of refactoring in adjacent lines.
* fixes 1 test and 1 benchmark that depended on PIPE being constant.

PR-URL: nodejs#14168
Fixes: nodejs#14128
Reviewed-By: James M Snell <[email protected]>
addaleax pushed a commit that referenced this issue Jul 18, 2017
* includes a tiny bit of refactoring in adjacent lines.
* fixes 1 test and 1 benchmark that depended on PIPE being constant.

PR-URL: #14168
Fixes: #14128
Reviewed-By: James M Snell <[email protected]>
Fishrock123 pushed a commit that referenced this issue Jul 19, 2017
* includes a tiny bit of refactoring in adjacent lines.
* fixes 1 test and 1 benchmark that depended on PIPE being constant.

PR-URL: #14168
Fixes: #14128
Reviewed-By: James M Snell <[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. stream Issues and PRs related to the stream subsystem. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants