Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

test: mark test-child-process-fork-dgram as flaky #8274

Closed
wants to merge 1 commit into from

Conversation

mhdawson
Copy link
Member

Checklist

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

test

Description of change

mark test-child-process-fork-dgram as flaky until we investigate/resolve
#8271

@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label Aug 25, 2016
@mhdawson
Copy link
Member Author

@mscdex mscdex added child_process Issues and PRs related to the child_process subsystem. dgram Issues and PRs related to the dgram subsystem / UDP. labels Aug 25, 2016
@mhdawson
Copy link
Member Author

Again planning for a quick land since its simply marking a test as flaky and will help CI health.

@jbergstroem
Copy link
Member

Something went wrong with last ci. New run: https://ci.nodejs.org/job/node-test-commit/4770/

@mhdawson
Copy link
Member Author

CI ok except for ARM machine issue, so should be good to land.

@mhdawson
Copy link
Member Author

Heading out for today, if there are enough oks by tomorrow will and first thing when I get in.

@cjihrig
Copy link
Contributor

cjihrig commented Aug 26, 2016

LGTM

1 similar comment
@joaocgreis
Copy link
Member

LGTM

@mhdawson
Copy link
Member Author

landed as 02ed21a

@mhdawson mhdawson closed this Aug 26, 2016
mhdawson added a commit that referenced this pull request Aug 26, 2016
PR-URL: #8274
Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@mhdawson mhdawson self-assigned this Aug 26, 2016
@Fishrock123 Fishrock123 mentioned this pull request Sep 6, 2016
Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request Sep 8, 2016
PR-URL: nodejs#8274
Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Fishrock123 pushed a commit that referenced this pull request Sep 9, 2016
PR-URL: #8274
Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
MylesBorins pushed a commit that referenced this pull request Oct 10, 2016
PR-URL: #8274
Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
MylesBorins pushed a commit that referenced this pull request Oct 10, 2016
PR-URL: #8274
Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
rvagg pushed a commit that referenced this pull request Oct 18, 2016
PR-URL: #8274
Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
MylesBorins pushed a commit that referenced this pull request Oct 26, 2016
PR-URL: #8274
Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Oct 26, 2016
@mhdawson mhdawson deleted the fork-dgram branch March 15, 2017 21:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
child_process Issues and PRs related to the child_process subsystem. dgram Issues and PRs related to the dgram subsystem / UDP. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants