Merge pull request #35833 from apelisse/owners-pkg-controller

Automatic merge from submit-queue Curating Owners: pkg/controller cc @jsafrane @mikedanese @bprashanth @derekwaynecarr @thockin @saad-ali 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 perfectly: 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. Use PR https://github.com/kubernetes/kubernetes/pull/35715 as an example. 2. The pull-request is made editable, please edit the OWNERS file to add the names of people that should be reviewing code in the future in the **reviewers** section. You probably do NOT need to modify the **approvers** section. 3. Notify me if you want some OWNERS file to be removed. Being an approver or reviewer of a parent directory makes you a reviewer/approver of the subdirectories too, so not all OWNERS files may be necessary. 4. Please use ALIAS if you want to use the same list of people over and over again (don't hesitate to ask me for help, or use the pull-request above as an example)

Merge pull request #35833 from apelisse/owners-pkg-controller
f45e918b · Kubernetes Submit Queue · GitHub · 6c059cb9 · 8ff3182f · f45e918b
Commit f45e918b authored Dec 12, 2016 by Kubernetes Submit Queue Committed by GitHub Dec 12, 2016
24 changed files
--- a/pkg/controller/OWNERS
+++ b/pkg/controller/OWNERS
-assignees:
+approvers:
-  - bprashanth
+- bprashanth
-  - derekwaynecarr
+- derekwaynecarr
-  - mikedanese
+- mikedanese
+reviewers:
+- deads2k
--- a/pkg/controller/certificates/OWNERS
+++ b/pkg/controller/certificates/OWNERS
+reviewers:
+- deads2k
+- mikedanese
--- a/pkg/controller/daemon/OWNERS
+++ b/pkg/controller/daemon/OWNERS
+reviewers:
+- mikedanese
+- kargakis
+- lukaszo
--- a/pkg/controller/deployment/OWNERS
+++ b/pkg/controller/deployment/OWNERS
+reviewers:
+- janetkuo
+- nikhiljindal
+- kargakis
+- mfojtik
--- a/pkg/controller/disruption/OWNERS
+++ b/pkg/controller/disruption/OWNERS
+reviewers:
+- mml
+- davidopp
+- kargakis
+- mwielgus
--- a/pkg/controller/endpoint/OWNERS
+++ b/pkg/controller/endpoint/OWNERS
+reviewers:
+- bprashanth
+- bowei 
+- MrHohn
+- thockin
+- mrubin
--- a/pkg/controller/garbagecollector/OWNERS
+++ b/pkg/controller/garbagecollector/OWNERS
+reviewers:
+- caesarxuchao
+- lavalamp
+- deads2k
--- a/pkg/controller/informers/OWNERS
+++ b/pkg/controller/informers/OWNERS
+reviewers:
+- derekwaynecarr
+- caesarxuchao
+- wojtek-t
+- lavalamp
+- deads2k
--- a/pkg/controller/job/OWNERS
+++ b/pkg/controller/job/OWNERS
+reviewers:
+- erictune
+- soltysh
--- a/pkg/controller/namespace/OWNERS
+++ b/pkg/controller/namespace/OWNERS
+reviewers:
+- derekwaynecarr
+- smarterclayton
--- a/pkg/controller/node/OWNERS
+++ b/pkg/controller/node/OWNERS
+reviewers:
+- gmarek
+- smarterclayton
+- ingvagabund
+- aveshagarwal
--- a/pkg/controller/petset/OWNERS
+++ b/pkg/controller/petset/OWNERS
+reviewers:
+- foxish
+- bprashanth
+- smarterclayton
+- janetkuo
+- kargakis
--- a/pkg/controller/podautoscaler/OWNERS
+++ b/pkg/controller/podautoscaler/OWNERS
+reviewers:
+- DirectXMan12
+- mwielgus
+- piosz
+- jszczepkowski
+- fgrzadkowski
--- a/pkg/controller/podgc/OWNERS
+++ b/pkg/controller/podgc/OWNERS
+reviewers:
+- foxish
+- gmarek
--- a/pkg/controller/replicaset/OWNERS
+++ b/pkg/controller/replicaset/OWNERS
+reviewers:
+- caesarxuchao
+- kargakis 
+- lavalamp
+- bprashanth
--- a/pkg/controller/replication/OWNERS
+++ b/pkg/controller/replication/OWNERS
+reviewers:
+- caesarxuchao
+- kargakis 
+- lavalamp
+- bprashanth
--- a/pkg/controller/resourcequota/OWNERS
+++ b/pkg/controller/resourcequota/OWNERS
+reviewers:
+- derekwaynecarr
+- deads2k
--- a/pkg/controller/route/OWNERS
+++ b/pkg/controller/route/OWNERS
+reviewers:
+- gmarek
+- wojtek-t
--- a/pkg/controller/scheduledjob/OWNERS
+++ b/pkg/controller/scheduledjob/OWNERS
+reviewers:
+- erictune
+- janetkuo
+- soltysh
--- a/pkg/controller/service/OWNERS
+++ b/pkg/controller/service/OWNERS
+reviewers:
+- bprashanth
+- bowei 
+- MrHohn
+- thockin
+- mrubin
--- a/pkg/controller/serviceaccount/OWNERS
+++ b/pkg/controller/serviceaccount/OWNERS
+reviewers:
+- liggitt
--- a/pkg/controller/volume/OWNERS
+++ b/pkg/controller/volume/OWNERS
+reviewers:
+- saad-ali
+- jsafrane
+- jingxu97
+- pmorie
+- justinsb
+- rootfs
--- a/pkg/controller/volume/attachdetach/OWNERS
+++ b/pkg/controller/volume/attachdetach/OWNERS
-assignees:
+approvers:
-  - saad-ali
+- saad-ali
--- a/pkg/controller/volume/persistentvolume/OWNERS
+++ b/pkg/controller/volume/persistentvolume/OWNERS
-assignees:
+approvers:
-  - jsafrane
+- jsafrane
-  - saad-ali
+- saad-ali
-  - thockin
+- thockin