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

Add an issue template for maintainer onboarding #1924

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

svrnm
Copy link
Member

@svrnm svrnm commented Feb 6, 2024

This PR introduces a dedicated issue template to elect and onboard a new maintainer.

Preview: https://github.com/svrnm/community/issues/new?assignees=&labels=area%2Fgithub-membership&projects=&template=maintainer-onboarding.yml&title=Onboard+%3CGitHub+handle%3E+as+%3CSIG%2FWG+FOO%3E+maintainer

Problem

the current process to elect maintainers has never(?) been applied, and since the list of maintainers & approvers lives mainly in the SIG repos now ("The list of active members (both "approvers" and "maintainers") for the can be found in the README file") it also does not match the requirements anymore.

Unfortunately, by only updating those files in the repo (or updating the github teams) the community does not hear about new maintainers stepping up. This makes it hard for cross-functional SIGS (Comms, EUWG, security, demo) to know who the maintainers are and also the TC&GC has no full visiblity into that, making it hard to judge from the outside if a SIG has a healthy set of maintainers. Finally, we (the OpenTelemetry community) should treat an individual stepping up as a maintainer as a highlight, that should be visible and recognized.

Solution

By introducing an onboarding issue for maintainers we can address all those needs, and at the same time also accomplish a few additional things, like having checklists for existing maintainers and new maintainers, or being able to introduce same additional rituals (e.g. introducing an informal chat between a new maintainer and a GC member to welcome them)

Please, take a look at this PR. I added a lot of additional features, that may not be wanted, so I expect this file to shrink, and there may be things I missed, so it might grow.

Other details

  • Similar to this we may consider introducing onboarding for other community roles (approvers), because I also think the community should know about them to have a chance to recognize the individuals.
  • If we implement Manage membership thru IaC #1793 we could go back to the "PR process" we have right now, but I argue this should simply become an item on that checklist.
  • Thanks to @danielgblanco for creating the GC member onboarding issue template, which was really helpful to get this one done.

.github/ISSUE_TEMPLATE/maintainer-onboarding.yml Outdated Show resolved Hide resolved
type: checkboxes
attributes:
label: Maintainer Onboarding Tasks
description: |
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this mention that these are to be done by a current maintainer?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, this needs to be clearer

type: checkboxes
attributes:
label: Maintainer Election
description: |
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this point back to the Becoming a Maintainer part of the docs rather than repeating the election process?

Copy link
Member Author

Choose a reason for hiding this comment

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

good point! Let me fix that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants