Skip to content

test-cluster-disconnect-handles hanging in background on freebsd #6988

Description

@rvagg

found 10 instances of this running on one of the freebsd CI machines, e.g:

iojs          97199   0.0  0.1 620772  1772  -  I    Sat09PM      0:00.09 /usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/out/Release/node --debug=12346. /usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/test/parallel/test-cluster-disconnect-handles.js

Activity

rvagg commented on May 26, 2016

@rvagg
MemberAuthor

@nodejs/testing

added
testIssues and PRs related to Node.js core tests and test infrastructure.
clusterIssues and PRs related to the cluster subsystem.
on May 26, 2016

Trott commented on May 26, 2016

@Trott
Member

If nothing is jumping out at anyone in the code, maybe try running this test 10K times on freebsd in the stress test job and see if it fails at some point (and see if that failure causes subsequent runs to fail too). And, of course, fix whatever is causing the first failure.

I guess in the time it took me to type that out, I could have just kicked off the job. So here it is: https://ci.nodejs.org/job/node-stress-single-test/744/nodes=freebsd102-64/console

cjihrig commented on May 26, 2016

@cjihrig
Contributor

The stress test came back green. Are there any processes left behind?

Trott commented on May 26, 2016

@Trott
Member

Expanding scope a bit, judging from failures like https://ci.nodejs.org/job/node-test-commit-arm/3454/nodes=armv7-ubuntu1404/consoleFull it seems that some test somewhere is not cleaning up after itself reliably, even when it succeeds. I imagine it would manifest as a stray node process running and keeping the port open, resulting in the plethora of EADDRINUSE errors. I imagine this is a likely source of this issue. Not sure of an easy way to identify the problematic test, though. EDIT: Oh, actually, maybe this is the culprit test...

added a commit that references this issue on May 27, 2016
added a commit that references this issue on May 30, 2016

cjihrig commented on Jun 1, 2016

@cjihrig
Contributor

@Trott, @rvagg can this one be closed? The issues with the CI from last week seem to be cleared up.

Trott commented on Jun 1, 2016

@Trott
Member

I think it can be closed.

cjihrig commented on Jun 1, 2016

@cjihrig
Contributor

OK. Closing. Please reopen if you think it needs to be investigated further.

added a commit that references this issue on Jun 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    clusterIssues and PRs related to the cluster subsystem.testIssues and PRs related to Node.js core tests and test infrastructure.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions