Merge pull request #35715 from apelisse/owners-pkg-kubectl

Automatic merge from submit-queue

Curating Owners: pkg/kubectl

In an effort to expand the existing pool of reviewers and establish a
two-tiered review process (first someone lgtms and then someone
experienced in the project approves), we are adding new reviewers to
existing owners files.
## If You Care About the Process:

We did this by algorithmically figuring out who’s contributed code to
the project and in what directories.  Unfortunately, that doesn’t work
well: people that have made mechanical code changes (e.g change the
copyright header across all directories) end up as reviewers in lots of
places.

Instead of using pure commit data, we generated an excessively large
list of reviewers and pruned based on all time commit data, recent
commit data and review data (number of PRs commented on).

At this point we have a decent list of reviewers, but it needs one last
pass for fine tuning.
## TLDR:

As an owner of a sig/directory and a leader of the project, here’s what
we need from you:
1. Comment on the pull-request explaining the changes you want. I'll
   take care of doing them (unless you want to modify the pull-request
   yourself).
2. OWNERS files in subdirectory can be removed if they are redundant with
   top-directory files.
3. Please use the ALIAS feature if you want to re-use a list of persons
   in different files.
This commit is contained in:
Kubernetes Submit Queue
2016-11-03 15:50:29 -07:00
committed by GitHub
3 changed files with 34 additions and 7 deletions

26
OWNERS_ALIASES Normal file
View File

@@ -0,0 +1,26 @@
aliases:
sig-cli-maintainers:
- adohe
- brendandburns
- deads2k
- fabianofranz
- janetkuo
- pwittrock
- smarterclayton
sig-cli:
- adohe
- deads2k
- derekwaynecarr
- dims
- dshulyak
- eparis
- ericchiang
- fabianofranz
- ghodss
- kargakis
- pwittrock
- rootfs
- smarterclayton
- soltysh
- sttts
- ymqytw

4
cmd/kubectl/OWNERS Normal file
View File

@@ -0,0 +1,4 @@
approvers:
- sig-cli-maintainers
reviewers:
- sig-cli

View File

@@ -1,7 +1,4 @@
assignees:
- brendandburns
- deads2k
- janetkuo
- jlowdermilk
- pwittrock
- smarterclayton
approvers:
- sig-cli-maintainers
reviewers:
- sig-cli