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

test: move watch mode tests to sequential #44610

Closed
wants to merge 1 commit into from

Conversation

MoLow
Copy link
Member

@MoLow MoLow commented Sep 12, 2022

following #44366 (comment)

@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 Sep 12, 2022
@MoLow MoLow added the fast-track PRs that do not need to wait for 48 hours to land. label Sep 12, 2022
@github-actions
Copy link
Contributor

Fast-track has been requested by @MoLow. Please 👍 to approve.

@MoLow MoLow added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label Sep 12, 2022
@MoLow MoLow added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 12, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 12, 2022
@nodejs-github-bot
Copy link
Collaborator

@MoLow MoLow added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Sep 12, 2022
@nodejs-github-bot nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Sep 12, 2022
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/44610
✔  Done loading data for nodejs/node/pull/44610
----------------------------------- PR info ------------------------------------
Title      test: move watch mode tests to sequential (#44610)
Author     Moshe Atlow  (@MoLow)
Branch     MoLow:move-watch-mode-test-to-sequential -> nodejs:main
Labels     test, flaky-test, fast-track, author ready, needs-ci
Commits    1
 - test: move watch mode tests to sequential
Committers 1
 - Moshe Atlow 
PR-URL: https://github.com/nodejs/node/pull/44610
Reviewed-By: Colin Ihrig 
Reviewed-By: Nitzan Uziely 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/44610
Reviewed-By: Colin Ihrig 
Reviewed-By: Nitzan Uziely 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Mon, 12 Sep 2022 10:20:34 GMT
   ✔  Approvals: 2
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/44610#pullrequestreview-1104128890
   ✔  - Nitzan Uziely (@linkgoron): https://github.com/nodejs/node/pull/44610#pullrequestreview-1104258489
   ℹ  This PR is being fast-tracked
   ✖  The fast-track request requires at least two collaborators' approvals (👍).
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2022-09-12T13:50:16Z: https://ci.nodejs.org/job/node-test-pull-request/46548/
- Querying data for job/node-test-pull-request/46548/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/3040106060

@MoLow MoLow added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Sep 12, 2022
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 12, 2022
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/44610
✔  Done loading data for nodejs/node/pull/44610
----------------------------------- PR info ------------------------------------
Title      test: move watch mode tests to sequential (#44610)
Author     Moshe Atlow  (@MoLow)
Branch     MoLow:move-watch-mode-test-to-sequential -> nodejs:main
Labels     test, flaky-test, fast-track, author ready, needs-ci
Commits    1
 - test: move watch mode tests to sequential
Committers 1
 - Moshe Atlow 
PR-URL: https://github.com/nodejs/node/pull/44610
Reviewed-By: Colin Ihrig 
Reviewed-By: Nitzan Uziely 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/44610
Reviewed-By: Colin Ihrig 
Reviewed-By: Nitzan Uziely 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Mon, 12 Sep 2022 10:20:34 GMT
   ✔  Approvals: 2
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/44610#pullrequestreview-1104128890
   ✔  - Nitzan Uziely (@linkgoron): https://github.com/nodejs/node/pull/44610#pullrequestreview-1104258489
   ℹ  This PR is being fast-tracked
   ✖  The fast-track request requires at least two collaborators' approvals (👍).
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2022-09-12T19:45:26Z: https://ci.nodejs.org/job/node-test-pull-request/46548/
- Querying data for job/node-test-pull-request/46548/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/3040437712

@nodejs-github-bot nodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 12, 2022
@MoLow
Copy link
Member Author

MoLow commented Sep 12, 2022

superseded by #44621

@MoLow MoLow closed this Sep 12, 2022
@MoLow MoLow deleted the move-watch-mode-test-to-sequential branch September 12, 2022 20:49
@MoLow MoLow added the watch-mode Issues and PRs related to watch mode label Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. fast-track PRs that do not need to wait for 48 hours to land. flaky-test Issues and PRs related to the tests with unstable failures on the CI. needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. watch-mode Issues and PRs related to watch mode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants