Add web-infra onboarding docs #17
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| # @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. | ||
|
|
||
| ## Tasks | ||
|
|
||
| Tasks prefixed by `Member:` are for the new member to complete. | ||
| Tasks prefixed by `Web Admin:` are for a member of `@nodejs/web-admins` to complete. | ||
|
|
||
| - [ ] Web Admin: Create an issue in [nodejs/web-team](https://github.com/nodejs/web-team) specifically for onboarding the member and include this list in it. | ||
|
There was a problem hiding this comment. Honestly think this might be better as an issue template and this doc just links to it, wdyt? There was a problem hiding this comment. I think the Member can do this in their "Add xyz to the list of MEMBERS" PR, and the team will check off each task before merging? |
||
| - Please link any additional issues or PRs relating to the member's onboarding to this issue for record keeping. | ||
| - [ ] Web Admin: Recommend member have some form of physical 2FA (i.e. Yubikey) or passkey enabled on their GitHub account, and that any SSH or GPG key attached to their GitHub account have passphrases. | ||
|
There was a problem hiding this comment. The Node.js Organization requires 2FA, so this isn't something we need to explicitly say. There was a problem hiding this comment. We do need to state this -- the org requires 2FA, this is a recommendation specifically to use a physical factor, which AIUI is not enforced at the org level. |
||
| - [ ] Web Admin: Recommend member sign their Git commits if not already doing so. | ||
|
There was a problem hiding this comment. I feel like the "Web Admin: Recommend" aren't really needed, and should be put as a separate section "# Recommendations for New Members" of this file? |
||
| - [ ] Web Admin: Add member to the [@nodejs/web-infra](https://github.com/orgs/nodejs/teams/web-infra) team. | ||
| - [ ] Member: Add yourself to the [membership list document](https://github.com/nodejs/web-team/blob/main/MEMBERS.md#nodejs-web-infra-team-nodejsweb-infra). | ||
| - [ ] Member: Open an issue in [nodejs/admin](https://github.com/nodejs/admin) requesting access to the Web Infra 1Password vault. | ||
| - [ ] Web Admin: Invite member to Node.js' Sentry account. | ||
| - [ ] Member: Enable physical or passkey 2FA on their Sentry account. | ||
flakey5 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| - [ ] Web Admin: Invite member to Node.js' Vercel account. | ||
| - [ ] Member: Enable physical or passkey 2FA on their Vercel account. | ||
|
|
||
| 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. | ||
| You may be granted access if needed, but will not be given it initially since we do not control it. | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I feel like we can just say "A web admin should ...", "The onboardee should ...". in each task