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: exclude tests failing on AIX for v4.x #9746

Closed
wants to merge 1 commit into from

Conversation

mhdawson
Copy link
Member

@mhdawson mhdawson commented Nov 22, 2016

Checklist
  • make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Description of change

Exclude tests that have been fixed in later versions but that fail in v4.x. Excluding these will reduce the noise when builds are run for older versions. The tests were excluded earlier as part of:

ecbeeeb

but that never made it back to v6.x and we don't need the full commit just what is included in this PR.

I noticed these as there were quite a few 4.x runs today leading to a lot red in the aix builds.

Exclude tests that have been fixed in later versions
but that fail in v4.x. Excluding these will reduce
the noise when builds are run for older versions.
@mhdawson
Copy link
Member Author

@nodejs/platform-aix

@addaleax
Copy link
Member

@mhdawson Have you seen #9733? (Sorry, maybe I should have indicated to wait for your opinion on that…)

@mhdawson
Copy link
Member Author

mhdawson commented Nov 22, 2016

Nope, just came across it now and was going to say "never mind" in this one. If its been backported then all the better. :)

@mhdawson mhdawson closed this Nov 22, 2016
@mscdex mscdex added test Issues and PRs related to the tests. aix Issues and PRs related to the AIX platform. labels Nov 23, 2016
@mhdawson mhdawson deleted the aix4flaky branch March 15, 2017 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aix Issues and PRs related to the AIX platform. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants