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

Adding maintainers to opensearch-project. #59

Merged
merged 29 commits into from
Apr 13, 2022
Merged
Changes from 2 commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
978e48c
Proposal: adding maintainers to opensearch-project.
dblock Mar 29, 2022
ec22ee3
Removed link to ADMINS.md, it's misleading because it's only in this …
dblock Mar 29, 2022
f97e0e3
Fix: typo.
dblock Mar 30, 2022
5ce9ebb
Added deliberate harmful actions.
dblock Mar 30, 2022
6f9ba7e
Added a note on less than 3 maintainers.
dblock Mar 30, 2022
b5a58c0
Replaced repo owner by admin.
dblock Mar 30, 2022
ff9c5c5
Consistently lowercase repo admin.
dblock Mar 30, 2022
609a79e
Adding a New Maintainer.
dblock Mar 30, 2022
4e9091f
Properly reflect that a nomation happened already.
dblock Mar 31, 2022
98858f2
Use a pull request instead of an additional issue.
dblock Mar 31, 2022
f4e8ba2
Remove an individual from the nomination example.
dblock Mar 31, 2022
7acc925
Linked admin responsibilities.
dblock Mar 31, 2022
7fd6c11
Document that the maintainer needs to reengage, not just intend to.
dblock Apr 1, 2022
b5d0835
One week to several weeks.
dblock Apr 5, 2022
f2e5328
Proposal and vote.
dblock Apr 5, 2022
6247990
Clarified the public aspect of the revote.
dblock Apr 5, 2022
ab99c7a
Don't be so prescriptive about inactivity.
dblock Apr 7, 2022
5e5316d
Replace that the account is not deactivated.
dblock Apr 7, 2022
9387240
Rollup changes.
dblock Apr 7, 2022
0851f1d
Minor language updates based on feedback.
dblock Apr 8, 2022
6947878
Remove public wait discussion period.
dblock Apr 8, 2022
6d68d52
Maintainers will be removed for violating code of coduct.
dblock Apr 8, 2022
9580d11
Updated engagement details.
dblock Apr 8, 2022
6a1fc43
Fix: duplicate message.
dblock Apr 8, 2022
7323884
Keep the ask private.
dblock Apr 11, 2022
a67ba9e
remove two week maximum
nknize Apr 12, 2022
0d79006
remove cr
nknize Apr 12, 2022
0e23acb
Use +1 / -1 everywhere.
dblock Apr 12, 2022
fed9887
Remove potential nominee.
dblock Apr 12, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
79 changes: 77 additions & 2 deletions MAINTAINERS.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,17 @@
- [Release Frequently](#release-frequently)
- [Promote Other Maintainers](#promote-other-maintainers)
- [Describe the Repo](#describe-the-repo)

- [Becoming a Maintainer](#becoming-a-maintainer)
- [Preparation](#preparation)
- [Interest](#interest)
- [Public Confirmation](#public-confirmation)
- [Maintainer Decision](#maintainer-decision)
- [Adding the New Maintainer](#adding-the-new-maintainer)
dblock marked this conversation as resolved.
Show resolved Hide resolved
- [Removing a Maintainer](#removing-a-maintainer)
- [Moving On](#moving-on)
- [Inactivity](#inactivity)
- [Negative Impact on the Project](#negative-impact-on-the-project)

dblock marked this conversation as resolved.
Show resolved Hide resolved
## Overview

This document explains who the maintainers are (see below), what they do in this repo, and how they should be doing it. If you're interested in contributing, see [CONTRIBUTING](CONTRIBUTING.md).
Expand Down Expand Up @@ -77,8 +87,73 @@ Make frequent project releases to the community.

### Promote Other Maintainers

Assist, add, and remove [MAINTAINERS](MAINTAINERS.md). Exercise good judgement, and propose high quality contributors to become co-maintainers.
Assist, add, and remove [MAINTAINERS](MAINTAINERS.md). Exercise good judgement, and propose high quality contributors to become co-maintainers. See [Becoming a Maintainer](#becoming-a-maintainer) for more information.

### Describe the Repo

Make sure the repo has a well-written, accurate, and complete description. See [opensearch-project/.github#38](https://github.com/opensearch-project/.github/issues/38) for some helpful tips to describe your repo.

## Becoming a Maintainer

You can become a maintainer by actively [contributing](CONTRIBUTING.md) to any project, and being nominated by an existing maintainer.
dblock marked this conversation as resolved.
Show resolved Hide resolved

### Nomination

Any current maintainer starts an e-mail thread (until we have a better mechanism, e-mails can usually be found via MAINTAINERS.md + DCO) with all other maintainers on that repository to discuss nomination using the template below. In order to be approved, at least 3 positive maintainer votes are necessary, and no vetoes. Any disagreements can be escalated to the repo [Admin](ADMINS.md).
dblock marked this conversation as resolved.
Show resolved Hide resolved

> I propose inviting Vacha Shah ([@vachashah](https://github.com/vachashah)) as a co-maintainer on [opensearch-project/OpenSearch](https://github.com/opensearch-project/OpenSearch).
dblock marked this conversation as resolved.
Show resolved Hide resolved
>
> Vacha has contributed a lot of automation across many repositories.
>
> - https://github.com/opensearch-project/OpenSearch/pull/877 — Added a broken link checker
> - https://github.com/opensearch-project/OpenSearch/issues/894 — Fixed broken links across the OpenSearch-Project
> - https://github.com/opensearch-project/opensearch-build/pull/17 — Updated labels across all OpenSearch repos
> - https://github.com/opensearch-project/opensearch-plugins/pull/70 — Updated plugin documentation
>
> Vacha has worked tenaciously on non-glamorous tasks, because they raise the overall Engineering bar for the entire project.

### Interest

Upon receiving at least 3 positive maintainer votes, and no vetoes, from other maintainers within a maximum period of 2 weeks, the nominating maintainer asks a potential nominee whether they might be interested in becoming a maintainer on the repository, which typically happens via e-mail obtained via DCO, but may also be a comment to a pull request, following a substantial contribution.
dblock marked this conversation as resolved.
Show resolved Hide resolved

> This is great work, Vacha! Would you be interested in becoming a [co-maintainer](https://github.com/opensearch-project/.github/blob/main/MAINTAINERS.md) on this project? I’d be honored to nominate you.
dblock marked this conversation as resolved.
Show resolved Hide resolved

Individuals accept the nomination by replying, or commenting, for example _"Thank you! I would love to."_
dblock marked this conversation as resolved.
Show resolved Hide resolved

### Public Confirmation

Upon receiving 3 positive maintainer votes, and no vetoes, from other maintainers, and after having confirmed interest with the nominee, the maintainer opens a public issue in order to make the vote visible to the community.
dblock marked this conversation as resolved.
Show resolved Hide resolved
dblock marked this conversation as resolved.
Show resolved Hide resolved

> _Content from the above nomination._
>
> Here is my +1.
>
> If you support this nomination, please add your +1 or a comment.
>
> If you do not support this nomination, we invite you to e-mail all/some of the current [project maintainers](https://github.com/opensearch-project/OpenSearch/blob/main/MAINTAINERS.md) in the next 2 weeks. We will keep your feedback private to this group and discuss offline. We will not publicly post the reasons for a decision not to add this co-maintainer, or publicize your name.
dblock marked this conversation as resolved.
Show resolved Hide resolved
>
> Anyone, including maintainers, can publicly add +1s as requested in the nomination issue if they support the nomination or contact the maintainers in private if they oppose. The issue stays open for 2 weeks.

### Maintainer Decision

If a positive decision has been reached, the nomination is accepted and the nominating maintainer can thus comment on the issue with _"maintainers has been added"_. If a decision cannot be reached for any reason, the nomination is rejected with _"a decision could not be reached"_ with no further explanation necessary.
dblock marked this conversation as resolved.
Show resolved Hide resolved

### Adding the New Maintainer

The Repo Admin adjusts the new maintainer’s permissions accordingly. The nominating maintainer opens a pull request to add the new maintainer to MAINTAINERS.md, and closes the nomination issue when that is merged and the individual has been given maintainer-level permissions.

## Removing a Maintainer

Removing a maintainer is a disruptive action that the community of maintainers should not undertake lightly. There are several reasons a maintainer might be removed from the project, such as inactivity, or taking actions that negatively impact the project.
dblock marked this conversation as resolved.
Show resolved Hide resolved
dblock marked this conversation as resolved.
Show resolved Hide resolved

### Moving On

There are plenty of reasons that might cause someone to want to take a step back or even a hiatus from a project. Existing maintainers can choose to leave the project at any time, with or without reason, by making a pull request to move themselves to the "Emeritus" section of MAINTAINERS.md, and asking an admin to remove their permissions.

### Inactivity
dblock marked this conversation as resolved.
Show resolved Hide resolved

If the maintainers on the project notice that another maintainer is no longer an active or visible member of the community, the first step will be to tell the repo owner to contact the inactive maintainer to check-in. If the inactive maintainer responds and intends to reengage, no action is required. If the maintainer does not respond after a week, or indicates they no longer wish to be a maintainer, the repo owner can remove the inactive maintainer’s permissions and make a pull request to add them to the "Emeritus" section of the MAINTAINERS.md.
dblock marked this conversation as resolved.
Show resolved Hide resolved

### Negative Impact on the Project
dblock marked this conversation as resolved.
Show resolved Hide resolved

Actions that negatively impact the project will be handled by the Admins, in coordination with other maintainers, in balance with the urgency of the issue. Examples would be [Code of Conduct](CODE_OF_CONDUCT.md) violations and security risks.
dblock marked this conversation as resolved.
Show resolved Hide resolved