Skip to content

Conversation

@mcollina
Copy link
Member

@mcollinamcollina commented Jul 23, 2022

Signed-off-by: Matteo Collina [email protected]

Ref: nodejs/reliability#329

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jul 23, 2022
@mcollinamcollina added request-ci Add this label to start a Jenkins CI on a PR. and removed test Issues and PRs related to the tests. needs-ci PRs that need a full CI run. labels Jul 23, 2022
Copy link
Member

@tniessentniessen left a comment

Choose a reason for hiding this comment

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

I think sequential/test-gc-http-client is also flaky (just like sequential/test-gc-http-client-timeout).

@mcollina
Copy link
MemberAuthor

Definitely. sequential/test-gc-http-client failed 7 times yesterday like this one.

I opened a PR at a time as it seems how it was done other times, but I'm also ok in massively adding all of them in one.

@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 23, 2022
@nodejs-github-bot
Copy link
Collaborator

@lpinca
Copy link
Member

It looks like the issue will be fixed by #43949.

@F3n67u
Copy link
Contributor

FYI, #43949 has merged to main branch and fixed test-gc-http-client-onerror test. So this pr could be closed.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@mcollina@nodejs-github-bot@lpinca@F3n67u@tniessen