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

Jump to bottom

test: mark test-cluster-shared-leak flaky #4162

Closed
wants to merge 1 commit into from
Closed

test: mark test-cluster-shared-leak flaky #4162

wants to merge 1 commit into from

Conversation

Trott
Copy link
Member

@Trott Trott commented Dec 5, 2015

test-cluster-shared-leak is flaky on Windows.

Ref: #3956

@Trott
test: mark test-cluster-shared-leak flaky
218aac9 
test-cluster-shared-leak is flaky on Windows.

Ref: nodejs#3956
@Trott Trott added windows Issues and PRs related to the Windows platform. test Issues and PRs related to the tests. labels Dec 5, 2015
@Trott
Copy link
Member Author

Trott commented Dec 5, 2015

Also took the opportunity to alphabetize the list of flaky tests on Windows.

@bnoordhuis
Copy link
Member

LGTM

2 similar comments
@r-52
Copy link
Contributor

r-52 commented Dec 5, 2015

LGTM

@JungMinu
Copy link
Member

JungMinu commented Dec 5, 2015

LGTM

@mscdex mscdex added the cluster Issues and PRs related to the cluster subsystem. label Dec 5, 2015
Trott added a commit that referenced this pull request Dec 6, 2015
@Trott @r-52
test: mark test-cluster-shared-leak flaky
afd5122 
test-cluster-shared-leak is flaky on Windows.

Refs: #3956

PR-URL: #4162
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
@r-52
Copy link
Contributor

r-52 commented Dec 6, 2015

Thanks @Trott - Landed in afd5122

@r-52 r-52 closed this Dec 6, 2015
@r-52 r-52 mentioned this pull request Dec 6, 2015
Closed
Trott added a commit that referenced this pull request Dec 8, 2015
@Trott @rvagg
test: mark test-cluster-shared-leak flaky
e84aeec 
test-cluster-shared-leak is flaky on Windows.

Refs: #3956

PR-URL: #4162
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
This was referenced Dec 8, 2015
Closed
Merged
@rvagg rvagg mentioned this pull request Dec 17, 2015
Closed
Trott added a commit that referenced this pull request Dec 29, 2015
@Trott
test: mark test-cluster-shared-leak flaky
78dc27b 
test-cluster-shared-leak is flaky on Windows.

Refs: #3956

PR-URL: #4162
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 19, 2016
@Trott
test: mark test-cluster-shared-leak flaky
6f4ab1d 
test-cluster-shared-leak is flaky on Windows.

Refs: #3956

PR-URL: #4162
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Jan 19, 2016
Merged
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
@Trott
test: mark test-cluster-shared-leak flaky
f4351d2 
test-cluster-shared-leak is flaky on Windows.

Refs: nodejs#3956

PR-URL: nodejs#4162
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
This was referenced Jul 23, 2021
Open
Open
Open
This was referenced Jul 31, 2021
Open
Open
Open
@snyk-bot snyk-bot mentioned this pull request Aug 12, 2021
Open
@Trott Trott deleted the double-secret-probation branch January 13, 2022 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cluster Issues and PRs related to the cluster subsystem. test Issues and PRs related to the tests. windows Issues and PRs related to the Windows platform.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@Trott @bnoordhuis @r-52 @JungMinu @mscdex @jasnell @MylesBorins