diff options
| author | Kubernetes Prow Robot <k8s-ci-robot@users.noreply.github.com> | 2019-05-06 15:47:39 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-05-06 15:47:39 -0700 |
| commit | 4aedcd41c07efc6e68f4bb48dc665bbdc00f1136 (patch) | |
| tree | 5574442ff5be2c66c87c41d52abaec6a815c2149 | |
| parent | 56a4fad9f53c452f2c63bbdeb9543b9032cb6cf0 (diff) | |
| parent | 4bda78d39187a9e551f06cdcedb64bb492cdafea (diff) | |
Merge pull request #3627 from mrbobbytables/add-emeritus-owners
Add emeritus to owners guidelines
| -rw-r--r-- | contributors/guide/owners.md | 28 |
1 files changed, 26 insertions, 2 deletions
diff --git a/contributors/guide/owners.md b/contributors/guide/owners.md index e61bdd17..e18c6224 100644 --- a/contributors/guide/owners.md +++ b/contributors/guide/owners.md @@ -11,7 +11,7 @@ inspired [GitHub's CODEOWNERS files](https://help.github.com/articles/about-code The velocity of a project that uses code review is limited by the number of people capable of reviewing code. The quality of a person's code review is limited by their familiarity with the code under review. Our goal is to address both of these concerns through the prudent use and maintenance -of OWNERS files +of OWNERS files. ## OWNERS spec @@ -86,6 +86,28 @@ Instead, set a `.*` key inside `filters` (as shown in the previous example). **WARNING**: The `approve` plugin [does not currently respect `filters`][test-infra-7690]. Until that is fixed, `filters` should only be used for the `labels` key (as shown in the above example). + +#### Emeritus + +It is inevitable, but there are times when someone may shift focuses, change jobs or step away from +a specific area in the project for a time. These people may be domain experts over certain areas +of the codebase, but can no longer dedicate the time needed to handle the responsibilities of +reviewing and approving changes. They are encouraged to add themselves as an _"emeritus"_ approver +under the `emeritus_approvers` key. + +GitHub usernames listed under the `emeritus_approvers` key can no longer approve code (use the +`/approve` command) and will be ignored by prow for assignment. However, it can still be referenced +by a person looking at the OWNERS file for a possible second or more informed opinion. + +When a contributor returns to being more active in that area, they may be promoted back to a +regular approver at the discretion of the current approvers. + +```yaml +emeritus_approvers: +- david # 2018-05-02 +- emily # 2019-01-05 +``` + ### OWNERS_ALIASES Each repo may contain at its root an OWNERS_ALIAS file. @@ -265,7 +287,9 @@ pieces of prow are used to implement the code review process above. OWNERS files should be regularly maintained. -We encourage people to self-nominate or self-remove from OWNERS files via PR's. Ideally in the future we could use metrics-driven automation to assist in this process. +We encourage people to self-nominate, self-remove or switch to [emeritus](#emeritus) from OWNERS +files via PR's. Ideally in the future we could use metrics-driven automation to assist in this +process. We should strive to: |
