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

test: mark wpt/test-user-timing test flaky by mhdawson · Pull Request #41203 · nodejs/node · GitHub

/ node Public

test: mark wpt/test-user-timing test flaky - #41203

Closed
mhdawson wants to merge 2 commits into
nodejs:masterfrom
mhdawson:mark-flaky
Closed

test: mark wpt/test-user-timing test flaky#41203
mhdawson wants to merge 2 commits into
nodejs:masterfrom
mhdawson:mark-flaky

Conversation

Copy link
Copy Markdown
Member
  • The RH team had a team day looking at helping move
    the CI closer to green. This is one of the flaky tests
    with the most reported failures and has been open
    since October. Marking flaky.

Signed-off-by: Michael Dawson mdawson@devrus.com

- The RH team had a team day looking at helping move
  the CI closer to green. This is one of the flaky tests
  with the most reported failures and has been open
  since October. Marking flaky.

Signed-off-by: Michael Dawson <mdawson@devrus.com>
nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Dec 16, 2021

lpinca commented Dec 16, 2021

Copy link
Copy Markdown
Member

I think it makes sense to mark it flaky on all platforms.

Copy link
Copy Markdown
Member Author

@lpinca updated.

aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 19, 2021
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 19, 2021

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/41203
✔  Done loading data for nodejs/node/pull/41203
----------------------------------- PR info ------------------------------------
Title      test: mark wpt/test-user-timing test flaky (#41203)
Author     Michael Dawson  (@mhdawson)
Branch     mhdawson:mark-flaky -> nodejs:master
Labels     test, needs-ci
Commits    2
 - test: mark wpt/test-user-timing test flaky
 - Update wpt.status
Committers 2
 - Michael Dawson 
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/41203
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Adrian Estrada 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/41203
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Adrian Estrada 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 16 Dec 2021 19:05:42 GMT
   ✔  Approvals: 3
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/41203#pullrequestreview-834530822
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/41203#pullrequestreview-835346797
   ✔  - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/41203#pullrequestreview-835411926
   ✔  Last GitHub Actions successful
   ✖  No Jenkins CI runs detected
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1598856323

nodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Dec 19, 2021

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Member Author

Landed in 555e7f9

mhdawson closed this Dec 22, 2021
mhdawson added a commit that referenced this pull request Dec 22, 2021
- The RH team had a team day looking at helping move
  the CI closer to green. This is one of the flaky tests
  with the most reported failures and has been open
  since October. Marking flaky.

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41203
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
targos pushed a commit that referenced this pull request Jan 14, 2022
- The RH team had a team day looking at helping move
  the CI closer to green. This is one of the flaky tests
  with the most reported failures and has been open
  since October. Marking flaky.

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41203
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
- The RH team had a team day looking at helping move
  the CI closer to green. This is one of the flaky tests
  with the most reported failures and has been open
  since October. Marking flaky.

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41203
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
- The RH team had a team day looking at helping move
  the CI closer to green. This is one of the flaky tests
  with the most reported failures and has been open
  since October. Marking flaky.

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41203
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
danielleadams mentioned this pull request Feb 1, 2022
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

commit-queue-failed An error occurred while landing this pull request using GitHub Actions. 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.

6 participants


Back | FazBrowse Home | New Git URL