-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
doc: security release announcement safety check #40333
Conversation
This happened today: https://www.githubstatus.com/incidents/bdbzpz7qxmbx. |
@@ -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/. |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
Github Actions could have an outage and we should abort the publishing of the PR if they are.
fe0ae0f
to
99be55b
Compare
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/.ncuhttps://github.com/nodejs/node/actions/runs/1315788152 |
Landed in 1fc1b0f...a3bfcb0 |
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>
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>
Github Actions could have an outage and we should abort the publishing
of the PR if they are.