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: make cluster tests more time tolerant #2891

Closed
wants to merge 1 commit into from

Conversation

mhdawson
Copy link
Member

Port nodejs/node-v0.x-archive@f3f4e28
to master, updating to guard changes for AIX as requestd

Port nodejs/node-v0.x-archive@f3f4e28
to master, updating to guard changes for AIX as requestd
@Trott
Copy link
Member

Trott commented Sep 15, 2015

LGTM. (We don't have AIX in the current CI, do we?)

@Fishrock123
Copy link
Contributor

LGTM

@mscdex mscdex added cluster Issues and PRs related to the cluster subsystem. test Issues and PRs related to the tests. labels Sep 15, 2015
@mhdawson
Copy link
Member Author

Thanks for the quick reviews. We don't have AIX in the CI yet. I'm working to try and get AIX machines but it seems that its not something that is commonly available to OS projects so I'm trying to figure out how we make one available.

@mhdawson mhdawson self-assigned this Sep 16, 2015
@mhdawson
Copy link
Member Author

@mhdawson
Copy link
Member Author

Some failures in CI but not related to test being changed and seen in other unrelated CI runs so not related to the change being made. Will land.

mhdawson added a commit that referenced this pull request Sep 17, 2015
Port nodejs/node-v0.x-archive@f3f4e28
to master, updating to guard changes for AIX as requested

PR-URL: #2891
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
@mhdawson
Copy link
Member Author

Landed as 2853f98

@Trott Trott closed this Sep 17, 2015
mhdawson added a commit that referenced this pull request Sep 20, 2015
Port nodejs/node-v0.x-archive@f3f4e28
to master, updating to guard changes for AIX as requested

PR-URL: #2891
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
@rvagg rvagg mentioned this pull request Sep 22, 2015
@MylesBorins
Copy link
Contributor

landed in lts-v4.x-staging as 71b5d80

@ncopa
Copy link
Contributor

ncopa commented Feb 3, 2016

This issue affects Alpine Linux (musl libc) also and not only AIX.

@mhdawson mhdawson deleted the aixcluster branch May 9, 2016 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cluster Issues and PRs related to the cluster subsystem. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants