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

doc: security release announcement safety check #40333

Closed

Conversation

mcollina
Copy link
Member

@mcollina mcollina commented Oct 5, 2021

Github Actions could have an outage and we should abort the publishing
of the PR if they are.

@mcollina
Copy link
Member Author

mcollina commented Oct 5, 2021

@@ -35,6 +35,8 @@ information described.

## Announcement (one week in advance of the planned release)

* [ ] Verify that Github Actions are working as normal: https://www.githubstatus.com/.
Copy link
Contributor

@mscdex mscdex Oct 5, 2021

Choose a reason for hiding this comment

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

Should this be in a different section? Seems like verifying this would be more of a last minute thing, not a "one week in advance" thing?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is actually the last minute thing. We can't publish to the website if the Github Actions are down.

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Oct 5, 2021
doc/guides/security-release-process.md Outdated Show resolved Hide resolved
@nodejs-github-bot
Copy link
Collaborator

Github Actions could have an outage and we should abort the publishing
of the PR if they are.
@mcollina mcollina force-pushed the update-security-guide-check-outage branch from fe0ae0f to 99be55b Compare October 6, 2021 15:46
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@mcollina mcollina added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 7, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 7, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2021

Commit Queue failed
- Loading data for nodejs/node/pull/40333
✔  Done loading data for nodejs/node/pull/40333
----------------------------------- PR info ------------------------------------
Title      doc: security release announcement safety check (#40333)
Author     Matteo Collina  (@mcollina)
Branch     mcollina:update-security-guide-check-outage -> nodejs:master
Labels     doc
Commits    1
 - doc: security release announcement safety check
Committers 1
 - Matteo Collina 
PR-URL: https://github.com/nodejs/node/pull/40333
Reviewed-By: Tobias Nießen 
Reviewed-By: Luigi Pinca 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40333
Reviewed-By: Tobias Nießen 
Reviewed-By: Luigi Pinca 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Tue, 05 Oct 2021 15:19:55 GMT
   ✔  Approvals: 2
   ✔  - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/40333#pullrequestreview-771685309
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/40333#pullrequestreview-773162062
   ✖  This PR needs to wait 4 more hours to land
   ✔  Last GitHub Actions successful
   ℹ  Green GitHub Actions CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1315788152

@github-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Oct 7, 2021
@mcollina mcollina removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Oct 7, 2021
@mcollina mcollina added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 7, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 7, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2021

Landed in 1fc1b0f...a3bfcb0

@github-actions github-actions bot closed this Oct 7, 2021
nodejs-github-bot pushed a commit that referenced this pull request Oct 7, 2021
Github Actions could have an outage and we should abort the publishing
of the PR if they are.

PR-URL: #40333
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit to danielleadams/node that referenced this pull request Oct 8, 2021
Github Actions could have an outage and we should abort the publishing
of the PR if they are.

PR-URL: nodejs#40333
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants