Skip to content

Conversation

@sam-github
Copy link
Contributor

fix#1933
replaces #1934

passed all tests except arm7 and raspberry pi 1 and 2, they are still ongoing

@Olegas@trevnorris PTAL

note that I think there are ALSO bugs in cluster message passing... this test attempts to evade them by waiting for the socket to be definitively open... by waiting for data from the worker/server

@sam-githubsam-githubforce-pushed the pr-1934-redux branch 2 times, most recently from 43a34a9 to 01225dfCompareJune 11, 2015 21:44
@sam-github
Copy link
ContributorAuthor

@mscdexmscdex added cluster Issues and PRs related to the cluster subsystem. test Issues and PRs related to the tests. labels Jun 11, 2015
@jbergstroem
Copy link
Member

LGTM. Verified on a machine I could reproduce on prior to patch.

@shigeki
Copy link
Contributor

Timeout error of test-cluster-worker-wait-server-close.js on my FreeBSD machine was resolved with this patch.

@shigekishigeki mentioned this pull request Jun 12, 2015
@rvagg
Copy link
Member

LGTM

Wait for data to arrive from worker before doing a disconnect. Without this, whether the disconnect arrives at the worker before the master accepts and forwards the connection descriptor to the worker is a race.
@sam-github
Copy link
ContributorAuthor

Landed in a5987d4d5e21be04a6239579634fa8b19446d5ef

@jbergstroem
Copy link
Member

@sam-github I can't find this commit in the master branch. Github doesn't either. Confuserating.

@jbergstroem
Copy link
Member

$ git fetch -q iojs && git branch -a --contains a5987d4d5e21be04a6239579634fa8b19446d5ef error: no such commit a5987d4d5e21be04a6239579634fa8b19446d5ef

@jbergstroem
Copy link
Member

I think I figured out what's going on. You pushed it to your own master branch, but for some reason the generated github link thinks the commit also lives in the nodejs/io.js repo (which it doesn't).

@Olegas
Copy link
Contributor

@sam-github
Test is fine. Passing on my environment. Thanks!
But can't see your commit on master also (
Reopened #1933 just to not get things lost.

@shigekishigeki mentioned this pull request Jun 12, 2015
sam-github added a commit that referenced this pull request Jun 12, 2015
Wait for data to arrive from worker before doing a disconnect. Without this, whether the disconnect arrives at the worker before the master accepts and forwards the connection descriptor to the worker is a race. Reviewed-By: Sam Roberts <[email protected]> Reviewed-By: Johan Bergström <[email protected]> Reviewed-By: Rod Vagg <[email protected]> PR-URL: #1953Fixes: #1933Fixes: #1400
@sam-github
Copy link
ContributorAuthor

Landed in 03ce84d

Sorry for the confusion

@jbergstroem
Copy link
Member

Probably not related to this specific test, but part of the same problem: Two freebsd bots saw this issue in a recent run: https://jenkins-iojs.nodesource.com/job/iojs+any-pr+multi/842/

I couldn't find any trailing processes on either machine. Could it possibly be some other test opening it and now being closed by this fix?

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clusterIssues and PRs related to the cluster subsystem.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: test-cluster-worker-wait-server-close.js is hanging

6 participants

@sam-github@jbergstroem@shigeki@rvagg@Olegas@mscdex