FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: mark http-pipeline-flood flaky by Trott · Pull Request #3616 · nodejs/node · GitHub

/ node Public

test: mark http-pipeline-flood flaky - #3616

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:mark-http-pipeline-flood-flaky
Closed

test: mark http-pipeline-flood flaky#3616
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:mark-http-pipeline-flood-flaky

Conversation

test-http-pipeline-flood has been flaky on Windows for some time.
Hopefully, nodejs#2862 fixes it and
lands soon, but until then, let's mark it as flaky.
Trott added the test Issues and PRs related to the tests. label Oct 31, 2015

Trott commented Oct 31, 2015

Copy link
Copy Markdown
Member Author

mscdex added the http Issues or PRs related to the http subsystem. label Nov 1, 2015

rvagg commented Nov 1, 2015

Copy link
Copy Markdown
Member

/cc @indutny also #2862

Copy link
Copy Markdown
Member

LGTM

Trott commented Nov 2, 2015

Copy link
Copy Markdown
Member Author

Landed in 6d6bc5d

Trott closed this Nov 2, 2015
Trott added a commit that referenced this pull request Nov 2, 2015
test-http-pipeline-flood has been flaky on Windows for some time.
Hopefully, #2862 fixes it and
lands soon, but until then, let's mark it as flaky.

PR-URL: #3616
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Trott added a commit that referenced this pull request Nov 7, 2015
test-http-pipeline-flood has been flaky on Windows for some time.
Hopefully, #2862 fixes it and
lands soon, but until then, let's mark it as flaky.

PR-URL: #3616
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Fishrock123 mentioned this pull request Nov 11, 2015

Copy link
Copy Markdown
Contributor

Holding off on landing this until deciding if #3636 is going to lts

Copy link
Copy Markdown
Contributor

unmarking this from lts-watch as #3636 has landed in lts

Copy link
Copy Markdown
Contributor

@jasnell the test is no longer flakey due to the fix in #3636 removing LTS watch

Trott added a commit to Trott/io.js that referenced this pull request Jan 17, 2016
test-http-pipeline-flood has been flaky on Windows for some time.
Hopefully, nodejs#2862 fixes it and
lands soon, but until then, let's mark it as flaky.

PR-URL: nodejs#3616
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Trott deleted the mark-http-pipeline-flood-flaky branch January 13, 2022 22:29
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

http Issues or PRs related to the http subsystem. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL