From 0332c3c34690f9988b49285f0e144af2cc2b3dd1 Mon Sep 17 00:00:00 2001 From: Antoine Pelisse Date: Thu, 27 Oct 2016 11:38:57 -0700 Subject: [PATCH 1/3] Update OWNERS approvers and reviewers: pkg/kubectl --- pkg/kubectl/OWNERS | 115 +++++++++++++++++++++++++++++++-- pkg/kubectl/cmd/OWNERS | 82 +++++++++++++++++++++++ pkg/kubectl/metricsutil/OWNERS | 2 + pkg/kubectl/resource/OWNERS | 39 +++++++++++ pkg/kubectl/testing/OWNERS | 10 +++ 5 files changed, 241 insertions(+), 7 deletions(-) create mode 100755 pkg/kubectl/cmd/OWNERS create mode 100755 pkg/kubectl/metricsutil/OWNERS create mode 100755 pkg/kubectl/resource/OWNERS create mode 100755 pkg/kubectl/testing/OWNERS diff --git a/pkg/kubectl/OWNERS b/pkg/kubectl/OWNERS index 26e3e7540eb..c940d83c59d 100644 --- a/pkg/kubectl/OWNERS +++ b/pkg/kubectl/OWNERS @@ -1,7 +1,108 @@ -assignees: - - brendandburns - - deads2k - - janetkuo - - jlowdermilk - - pwittrock - - smarterclayton +approvers: +- brendandburns +- deads2k +- janetkuo +- jlowdermilk +- pwittrock +- smarterclayton +reviewers: +- alex-mohr +- asalkeld +- ashcrow +- aveshagarwal +- bgrant0607 +- bprashanth +- brendandburns +- caesarxuchao +- caseydavenport +- cjcullen +- david-mcmahon +- davidopp +- dchen1107 +- ddysher +- deads2k +- derekwaynecarr +- dims +- dshulyak +- eparis +- ericchiang +- erictune +- errordeveloper +- fabianofranz +- fabioy +- feihujiang +- feiskyer +- fgrzadkowski +- filbranden +- freehan +- ghodss +- gmarek +- goltermann +- hongchaodeng +- ingvagabund +- ivan4th +- janetkuo +- jbeda +- jdef +- jeffmendoza +- jianhuiz +- jimmidyson +- jlowdermilk +- johscheuer +- jsafrane +- jszczepkowski +- juanvallejo +- justinsb +- k82cn +- kargakis +- kevin-wangzefeng +- krousey +- lavalamp +- liggitt +- lixiaobing10051267 +- lojies +- lukaszo +- luxas +- m1093782566 +- madhusudancs +- markturansky +- matchstick +- mbohlool +- mfojtik +- mikedanese +- mml +- mqliang +- mwielgus +- ncdc +- nhlfr +- nikhiljindal +- ping035627 +- piosz +- pmorie +- pweil- +- pwittrock +- quinton-hoole +- resouer +- rjnagal +- roberthbailey +- rootfs +- saad-ali +- satnam6502 +- screeley44 +- sdminonne +- sjenning +- smarterclayton +- soltysh +- sttts +- therc +- thockin +- timothysc +- timstclair +- tmrts +- vishh +- vmarmol +- wojtek-t +- xiangpengzhao +- ymqytw +- yujuhong +- zmerlynn diff --git a/pkg/kubectl/cmd/OWNERS b/pkg/kubectl/cmd/OWNERS new file mode 100755 index 00000000000..9f6c0cb33ad --- /dev/null +++ b/pkg/kubectl/cmd/OWNERS @@ -0,0 +1,82 @@ +reviewers: +- asalkeld +- aveshagarwal +- bprashanth +- brendandburns +- caesarxuchao +- cjcullen +- david-mcmahon +- davidopp +- dchen1107 +- ddysher +- deads2k +- derekwaynecarr +- dims +- dshulyak +- eparis +- ericchiang +- erictune +- errordeveloper +- fabianofranz +- fabioy +- feihujiang +- feiskyer +- fgrzadkowski +- filbranden +- freehan +- ghodss +- goltermann +- hongchaodeng +- ingvagabund +- ivan4th +- janetkuo +- jbeda +- jdef +- jianhuiz +- jimmidyson +- jlowdermilk +- jszczepkowski +- juanvallejo +- justinsb +- k82cn +- kargakis +- kevin-wangzefeng +- krousey +- lavalamp +- liggitt +- lixiaobing10051267 +- lojies +- luxas +- madhusudancs +- markturansky +- matchstick +- mbohlool +- mfojtik +- mikedanese +- mml +- mqliang +- mwielgus +- ncdc +- nhlfr +- nikhiljindal +- ping035627 +- piosz +- pmorie +- pwittrock +- resouer +- rjnagal +- roberthbailey +- satnam6502 +- sdminonne +- sjenning +- smarterclayton +- soltysh +- sttts +- thockin +- timothysc +- timstclair +- tmrts +- wojtek-t +- ymqytw +- yujuhong +- zmerlynn diff --git a/pkg/kubectl/metricsutil/OWNERS b/pkg/kubectl/metricsutil/OWNERS new file mode 100755 index 00000000000..81919881ff3 --- /dev/null +++ b/pkg/kubectl/metricsutil/OWNERS @@ -0,0 +1,2 @@ +reviewers: +- deads2k diff --git a/pkg/kubectl/resource/OWNERS b/pkg/kubectl/resource/OWNERS new file mode 100755 index 00000000000..aa9e93154c7 --- /dev/null +++ b/pkg/kubectl/resource/OWNERS @@ -0,0 +1,39 @@ +reviewers: +- aveshagarwal +- brendandburns +- caesarxuchao +- david-mcmahon +- davidopp +- dchen1107 +- deads2k +- derekwaynecarr +- dims +- eparis +- fabianofranz +- feihujiang +- fgrzadkowski +- gmarek +- ivan4th +- janetkuo +- jeffmendoza +- jlowdermilk +- juanvallejo +- k82cn +- kargakis +- krousey +- lavalamp +- liggitt +- lixiaobing10051267 +- mbohlool +- mikedanese +- mwielgus +- nikhiljindal +- pmorie +- pweil- +- pwittrock +- roberthbailey +- sdminonne +- smarterclayton +- wojtek-t +- ymqytw +- yujuhong diff --git a/pkg/kubectl/testing/OWNERS b/pkg/kubectl/testing/OWNERS new file mode 100755 index 00000000000..4aad008762b --- /dev/null +++ b/pkg/kubectl/testing/OWNERS @@ -0,0 +1,10 @@ +reviewers: +- david-mcmahon +- erictune +- liggitt +- mbohlool +- mml +- smarterclayton +- soltysh +- timothysc +- wojtek-t From 28bdf8298bafb4d4b4116add479b3469d2d93ba4 Mon Sep 17 00:00:00 2001 From: Phillip Wittrock Date: Thu, 27 Oct 2016 13:55:37 -0700 Subject: [PATCH 2/3] Update OWNERS --- pkg/kubectl/OWNERS | 89 ++-------------------------------- pkg/kubectl/cmd/OWNERS | 76 ++++------------------------- pkg/kubectl/metricsutil/OWNERS | 24 ++++++++- pkg/kubectl/resource/OWNERS | 43 ++++++---------- pkg/kubectl/testing/OWNERS | 28 ++++++++--- 5 files changed, 70 insertions(+), 190 deletions(-) diff --git a/pkg/kubectl/OWNERS b/pkg/kubectl/OWNERS index c940d83c59d..29ba5931090 100644 --- a/pkg/kubectl/OWNERS +++ b/pkg/kubectl/OWNERS @@ -1,108 +1,25 @@ approvers: +- adohe - brendandburns - deads2k +- fabianofranz - janetkuo -- jlowdermilk - pwittrock - smarterclayton reviewers: -- alex-mohr -- asalkeld -- ashcrow -- aveshagarwal -- bgrant0607 -- bprashanth -- brendandburns -- caesarxuchao -- caseydavenport -- cjcullen -- david-mcmahon -- davidopp -- dchen1107 -- ddysher +- adohe - deads2k - derekwaynecarr - dims - dshulyak - eparis - ericchiang -- erictune -- errordeveloper - fabianofranz -- fabioy -- feihujiang -- feiskyer -- fgrzadkowski -- filbranden -- freehan - ghodss -- gmarek -- goltermann -- hongchaodeng -- ingvagabund -- ivan4th -- janetkuo -- jbeda -- jdef -- jeffmendoza -- jianhuiz -- jimmidyson -- jlowdermilk -- johscheuer -- jsafrane -- jszczepkowski -- juanvallejo -- justinsb -- k82cn - kargakis -- kevin-wangzefeng -- krousey -- lavalamp -- liggitt -- lixiaobing10051267 -- lojies -- lukaszo -- luxas -- m1093782566 -- madhusudancs -- markturansky -- matchstick -- mbohlool -- mfojtik -- mikedanese -- mml -- mqliang -- mwielgus -- ncdc -- nhlfr -- nikhiljindal -- ping035627 -- piosz -- pmorie -- pweil- - pwittrock -- quinton-hoole -- resouer -- rjnagal -- roberthbailey - rootfs -- saad-ali -- satnam6502 -- screeley44 -- sdminonne -- sjenning - smarterclayton - soltysh - sttts -- therc -- thockin -- timothysc -- timstclair -- tmrts -- vishh -- vmarmol -- wojtek-t -- xiangpengzhao - ymqytw -- yujuhong -- zmerlynn diff --git a/pkg/kubectl/cmd/OWNERS b/pkg/kubectl/cmd/OWNERS index 9f6c0cb33ad..7b58189ca92 100755 --- a/pkg/kubectl/cmd/OWNERS +++ b/pkg/kubectl/cmd/OWNERS @@ -1,82 +1,24 @@ -reviewers: -- asalkeld -- aveshagarwal -- bprashanth +approvers: +- adohe - brendandburns -- caesarxuchao -- cjcullen -- david-mcmahon -- davidopp -- dchen1107 -- ddysher +- deads2k +- fabianofranz +- pwittrock +- smarterclayton +reviewers: +- adohe - deads2k - derekwaynecarr - dims - dshulyak - eparis - ericchiang -- erictune -- errordeveloper - fabianofranz -- fabioy -- feihujiang -- feiskyer -- fgrzadkowski -- filbranden -- freehan - ghodss -- goltermann -- hongchaodeng -- ingvagabund -- ivan4th -- janetkuo -- jbeda -- jdef -- jianhuiz -- jimmidyson -- jlowdermilk -- jszczepkowski -- juanvallejo -- justinsb -- k82cn - kargakis -- kevin-wangzefeng -- krousey -- lavalamp -- liggitt -- lixiaobing10051267 -- lojies -- luxas -- madhusudancs -- markturansky -- matchstick -- mbohlool -- mfojtik -- mikedanese -- mml -- mqliang -- mwielgus -- ncdc -- nhlfr -- nikhiljindal -- ping035627 -- piosz -- pmorie - pwittrock -- resouer -- rjnagal -- roberthbailey -- satnam6502 -- sdminonne -- sjenning +- rootfs - smarterclayton - soltysh - sttts -- thockin -- timothysc -- timstclair -- tmrts -- wojtek-t - ymqytw -- yujuhong -- zmerlynn diff --git a/pkg/kubectl/metricsutil/OWNERS b/pkg/kubectl/metricsutil/OWNERS index 81919881ff3..7b58189ca92 100755 --- a/pkg/kubectl/metricsutil/OWNERS +++ b/pkg/kubectl/metricsutil/OWNERS @@ -1,2 +1,24 @@ -reviewers: +approvers: +- adohe +- brendandburns - deads2k +- fabianofranz +- pwittrock +- smarterclayton +reviewers: +- adohe +- deads2k +- derekwaynecarr +- dims +- dshulyak +- eparis +- ericchiang +- fabianofranz +- ghodss +- kargakis +- pwittrock +- rootfs +- smarterclayton +- soltysh +- sttts +- ymqytw diff --git a/pkg/kubectl/resource/OWNERS b/pkg/kubectl/resource/OWNERS index aa9e93154c7..7b58189ca92 100755 --- a/pkg/kubectl/resource/OWNERS +++ b/pkg/kubectl/resource/OWNERS @@ -1,39 +1,24 @@ -reviewers: -- aveshagarwal +approvers: +- adohe - brendandburns -- caesarxuchao -- david-mcmahon -- davidopp -- dchen1107 +- deads2k +- fabianofranz +- pwittrock +- smarterclayton +reviewers: +- adohe - deads2k - derekwaynecarr - dims +- dshulyak - eparis +- ericchiang - fabianofranz -- feihujiang -- fgrzadkowski -- gmarek -- ivan4th -- janetkuo -- jeffmendoza -- jlowdermilk -- juanvallejo -- k82cn +- ghodss - kargakis -- krousey -- lavalamp -- liggitt -- lixiaobing10051267 -- mbohlool -- mikedanese -- mwielgus -- nikhiljindal -- pmorie -- pweil- - pwittrock -- roberthbailey -- sdminonne +- rootfs - smarterclayton -- wojtek-t +- soltysh +- sttts - ymqytw -- yujuhong diff --git a/pkg/kubectl/testing/OWNERS b/pkg/kubectl/testing/OWNERS index 4aad008762b..7b58189ca92 100755 --- a/pkg/kubectl/testing/OWNERS +++ b/pkg/kubectl/testing/OWNERS @@ -1,10 +1,24 @@ +approvers: +- adohe +- brendandburns +- deads2k +- fabianofranz +- pwittrock +- smarterclayton reviewers: -- david-mcmahon -- erictune -- liggitt -- mbohlool -- mml +- adohe +- deads2k +- derekwaynecarr +- dims +- dshulyak +- eparis +- ericchiang +- fabianofranz +- ghodss +- kargakis +- pwittrock +- rootfs - smarterclayton - soltysh -- timothysc -- wojtek-t +- sttts +- ymqytw From 91393b5123602fc87838f67b3b2d75765020c3fb Mon Sep 17 00:00:00 2001 From: Antoine Pelisse Date: Fri, 28 Oct 2016 14:04:20 -0700 Subject: [PATCH 3/3] Move pkg/kubectl reviewers into an alias --- OWNERS_ALIASES | 26 ++++++++++++++++++++++++++ cmd/kubectl/OWNERS | 4 ++++ pkg/kubectl/OWNERS | 25 ++----------------------- pkg/kubectl/cmd/OWNERS | 24 ------------------------ pkg/kubectl/metricsutil/OWNERS | 24 ------------------------ pkg/kubectl/resource/OWNERS | 24 ------------------------ pkg/kubectl/testing/OWNERS | 24 ------------------------ 7 files changed, 32 insertions(+), 119 deletions(-) create mode 100644 OWNERS_ALIASES create mode 100644 cmd/kubectl/OWNERS delete mode 100755 pkg/kubectl/cmd/OWNERS delete mode 100755 pkg/kubectl/metricsutil/OWNERS delete mode 100755 pkg/kubectl/resource/OWNERS delete mode 100755 pkg/kubectl/testing/OWNERS diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES new file mode 100644 index 00000000000..5dbcb4cfda0 --- /dev/null +++ b/OWNERS_ALIASES @@ -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 diff --git a/cmd/kubectl/OWNERS b/cmd/kubectl/OWNERS new file mode 100644 index 00000000000..ce699e9679a --- /dev/null +++ b/cmd/kubectl/OWNERS @@ -0,0 +1,4 @@ +approvers: +- sig-cli-maintainers +reviewers: +- sig-cli diff --git a/pkg/kubectl/OWNERS b/pkg/kubectl/OWNERS index 29ba5931090..ce699e9679a 100644 --- a/pkg/kubectl/OWNERS +++ b/pkg/kubectl/OWNERS @@ -1,25 +1,4 @@ approvers: -- adohe -- brendandburns -- deads2k -- fabianofranz -- janetkuo -- pwittrock -- smarterclayton +- sig-cli-maintainers reviewers: -- adohe -- deads2k -- derekwaynecarr -- dims -- dshulyak -- eparis -- ericchiang -- fabianofranz -- ghodss -- kargakis -- pwittrock -- rootfs -- smarterclayton -- soltysh -- sttts -- ymqytw +- sig-cli diff --git a/pkg/kubectl/cmd/OWNERS b/pkg/kubectl/cmd/OWNERS deleted file mode 100755 index 7b58189ca92..00000000000 --- a/pkg/kubectl/cmd/OWNERS +++ /dev/null @@ -1,24 +0,0 @@ -approvers: -- adohe -- brendandburns -- deads2k -- fabianofranz -- pwittrock -- smarterclayton -reviewers: -- adohe -- deads2k -- derekwaynecarr -- dims -- dshulyak -- eparis -- ericchiang -- fabianofranz -- ghodss -- kargakis -- pwittrock -- rootfs -- smarterclayton -- soltysh -- sttts -- ymqytw diff --git a/pkg/kubectl/metricsutil/OWNERS b/pkg/kubectl/metricsutil/OWNERS deleted file mode 100755 index 7b58189ca92..00000000000 --- a/pkg/kubectl/metricsutil/OWNERS +++ /dev/null @@ -1,24 +0,0 @@ -approvers: -- adohe -- brendandburns -- deads2k -- fabianofranz -- pwittrock -- smarterclayton -reviewers: -- adohe -- deads2k -- derekwaynecarr -- dims -- dshulyak -- eparis -- ericchiang -- fabianofranz -- ghodss -- kargakis -- pwittrock -- rootfs -- smarterclayton -- soltysh -- sttts -- ymqytw diff --git a/pkg/kubectl/resource/OWNERS b/pkg/kubectl/resource/OWNERS deleted file mode 100755 index 7b58189ca92..00000000000 --- a/pkg/kubectl/resource/OWNERS +++ /dev/null @@ -1,24 +0,0 @@ -approvers: -- adohe -- brendandburns -- deads2k -- fabianofranz -- pwittrock -- smarterclayton -reviewers: -- adohe -- deads2k -- derekwaynecarr -- dims -- dshulyak -- eparis -- ericchiang -- fabianofranz -- ghodss -- kargakis -- pwittrock -- rootfs -- smarterclayton -- soltysh -- sttts -- ymqytw diff --git a/pkg/kubectl/testing/OWNERS b/pkg/kubectl/testing/OWNERS deleted file mode 100755 index 7b58189ca92..00000000000 --- a/pkg/kubectl/testing/OWNERS +++ /dev/null @@ -1,24 +0,0 @@ -approvers: -- adohe -- brendandburns -- deads2k -- fabianofranz -- pwittrock -- smarterclayton -reviewers: -- adohe -- deads2k -- derekwaynecarr -- dims -- dshulyak -- eparis -- ericchiang -- fabianofranz -- ghodss -- kargakis -- pwittrock -- rootfs -- smarterclayton -- soltysh -- sttts -- ymqytw