Skip to content

Conversation

everett1992
Copy link
Contributor

test-domain-dep0097.js was sporadically failing because it is a parallel test and uses opens a default inspector port.

This commit changes to bind to a random free port

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels May 24, 2022
test-domain-dep0097.js was sporadically failing because it is a parallel
test and uses opens a default inspector port.

This commit changes to bind to a random free port
@tniessen tniessen added the request-ci Add this label to start a Jenkins CI on a PR. label May 26, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 26, 2022
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

mhdawson pushed a commit that referenced this pull request Jun 13, 2022
test-domain-dep0097.js was sporadically failing because it is a parallel
test and uses opens a default inspector port.

This commit changes to bind to a random free port

PR-URL: #43199
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@mhdawson
Copy link
Member

Landed in 5ca2ab1

@mhdawson mhdawson closed this Jun 13, 2022
danielleadams pushed a commit that referenced this pull request Jun 14, 2022
test-domain-dep0097.js was sporadically failing because it is a parallel
test and uses opens a default inspector port.

This commit changes to bind to a random free port

PR-URL: #43199
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@danielleadams danielleadams mentioned this pull request Jun 14, 2022
targos pushed a commit that referenced this pull request Jul 12, 2022
test-domain-dep0097.js was sporadically failing because it is a parallel
test and uses opens a default inspector port.

This commit changes to bind to a random free port

PR-URL: #43199
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
targos pushed a commit that referenced this pull request Jul 31, 2022
test-domain-dep0097.js was sporadically failing because it is a parallel
test and uses opens a default inspector port.

This commit changes to bind to a random free port

PR-URL: #43199
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
guangwong pushed a commit to noslate-project/node that referenced this pull request Oct 10, 2022
test-domain-dep0097.js was sporadically failing because it is a parallel
test and uses opens a default inspector port.

This commit changes to bind to a random free port

PR-URL: nodejs/node#43199
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants