-
-
Notifications
You must be signed in to change notification settings - Fork 4
Add web-infra onboarding docs #17
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,32 @@ | ||||||||||||||
# @nodejs/web-infra Onboarding | ||||||||||||||
|
||||||||||||||
This document is an outline of the tasks necessary to onboard a new member of the Web Infrastructure team (@nodejs/web-infra). | ||||||||||||||
|
||||||||||||||
Please note these tasks should only be performed after an individual's nomination has passed. | ||||||||||||||
To start the nomination process, see [GOVERNANCE.md](../GOVERNANCE.md). | ||||||||||||||
|
||||||||||||||
## Tasks | ||||||||||||||
|
||||||||||||||
- [ ] The nominee should open a pull request in [nodejs/web-team](https://github.com/nodejs/web-team) adding themselves to the [MEMBERS.md](https://github.com/nodejs/web-team/blob/main/MEMBERS.md#nodejs-web-infra-team-nodejsweb-infra) document. | ||||||||||||||
- [ ] Nominee should include this list in the pull request's body to keep track of what has been completed for their onboarding. The pull request should not be merged until the onboarding tasks have been completed. | ||||||||||||||
- [ ] Please link any additional issues or pull requests back to this one for record keeping purposes. | ||||||||||||||
Comment on lines
+10
to
+12
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
https://github.com/nodejs/web-team/blob/main/onboarding/README.md?plain=1#L10 |
||||||||||||||
- [ ] A Web Admin should add the nominee to the [@nodejs/web-infra](https://github.com/orgs/nodejs/teams/web-infra) team. | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||
- [ ] The nominee should open an issue in [nodejs/admin](https://github.com/nodejs/admin) requesting access to the Web Infra 1Password vault. | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||
- [ ] A Web Admin should invite the nominee to Node.js' Sentry account. | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||
- [ ] The nominee should enable physical or passkey 2FA on their Sentry account. | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||
- [ ] A Web Admin should invite the nominee to Node.js' Vercel account. | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||
- [ ] The nominee should enable physical or passkey 2FA on their Vercel account. | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This can go under |
||||||||||||||
- [ ] The nominee should open a pull request in [nodejs/email](https://github.com/nodejs/email) to add their email to the `nodejs-crowdin` email chain. | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||
- [ ] An OpenJS Slack Admin should add their Slack account to the `nodejs-website-team` team. | ||||||||||||||
|
||||||||||||||
### Recommendations | ||||||||||||||
|
||||||||||||||
- [ ] A Web Admin should recommend the nominee have some form of physical 2FA (i.e. Yubikey) or passkey enabled on their GitHub account | ||||||||||||||
- [ ] A Web Admin should recommend any SSH or GPG key attached to their GitHub account have passphrases or be stored on a physical 2FA device. | ||||||||||||||
- [ ] A Web Admin should recommend the nominee sign their Git commits. | ||||||||||||||
Comment on lines
+24
to
+26
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||
|
||||||||||||||
## Notes | ||||||||||||||
|
||||||||||||||
There currently is not a set level of access that @nodejs/web-infra members have on Cloudflare. | ||||||||||||||
|
||||||||||||||
Should you need additional access, it may be granted, however, access is not given by default since we do not control it. |
Uh oh!
There was an error while loading. Please reload this page.
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.
Let's make this uniform w/ https://github.com/nodejs/web-team/blob/main/onboarding/nodejs-website.md?plain=1