From 97055841b1c82bbd32f32a6fd5e2ca51e3068999 Mon Sep 17 00:00:00 2001 From: "Timothy St. Clair" Date: Mon, 7 Oct 2019 14:57:29 -0500 Subject: [PATCH] Audit of test/* OWNERS files --- test/OWNERS | 10 ++++------ test/conformance/OWNERS | 4 ++-- test/e2e/auth/OWNERS | 2 ++ test/e2e/autoscaling/OWNERS | 2 ++ test/e2e/cloud/OWNERS | 3 +-- test/e2e/common/OWNERS | 5 +++-- test/e2e/instrumentation/OWNERS | 4 +++- test/e2e/kubectl/OWNERS | 2 ++ test/e2e/lifecycle/OWNERS | 2 ++ test/e2e/network/OWNERS | 2 ++ test/e2e/node/OWNERS | 7 +++++-- test/e2e/scheduling/OWNERS | 2 ++ test/e2e/storage/OWNERS | 2 ++ test/e2e/windows/OWNERS | 5 ++++- test/e2e_node/OWNERS | 5 +++-- test/images/OWNERS | 7 +++---- test/instrumentation/OWNERS | 2 ++ test/integration/pods/OWNERS | 5 +++-- test/integration/statefulset/OWNERS | 8 -------- test/kubemark/OWNERS | 7 ++++--- test/typecheck/OWNERS | 3 ++- 21 files changed, 53 insertions(+), 36 deletions(-) delete mode 100644 test/integration/statefulset/OWNERS diff --git a/test/OWNERS b/test/OWNERS index 9e7f3489db2..176a39e4537 100644 --- a/test/OWNERS +++ b/test/OWNERS @@ -1,9 +1,7 @@ # See the OWNERS docs at https://go.k8s.io/owners reviewers: - - xichengliudui - neolit123 - - stevesloka - johnSchnake - SataQiu - draveness @@ -12,7 +10,6 @@ approvers: - andrewsykim - bowei # for test/e2e/{dns*,network}.go - cblecker - - rramkumar1 - MrHohn - deads2k - enisoc @@ -20,17 +17,14 @@ approvers: - liggitt - mikedanese - msau42 - - ncdc - oomichi - pwittrock # for test/e2e/kubectl.go - saad-ali - - shyamjvs - sig-testing-approvers - smarterclayton - soltysh - sttts - timothysc - - vishh emeritus_approvers: - enj - eparis @@ -43,6 +37,10 @@ emeritus_approvers: - marun - zmerlynn - MaciekPytel + - rramkumar1 + - ncdc + - vishh + - shyamjvs labels: - area/test diff --git a/test/conformance/OWNERS b/test/conformance/OWNERS index 6f1bc4708ab..bbcdd544d0e 100644 --- a/test/conformance/OWNERS +++ b/test/conformance/OWNERS @@ -2,13 +2,13 @@ # This is the owner of the test code. The test data itself is owned by sig-architecture. reviewers: - - mml - cheftako - oomichi approvers: - - mml - cheftako - spiffxp +emeritus_approvers: + - mml labels: - area/conformance - sig/architecture diff --git a/test/e2e/auth/OWNERS b/test/e2e/auth/OWNERS index 9d06868f54c..ea3cc14f408 100644 --- a/test/e2e/auth/OWNERS +++ b/test/e2e/auth/OWNERS @@ -12,3 +12,5 @@ approvers: - smarterclayton - sttts - tallclair +labels: +- sig/auth diff --git a/test/e2e/autoscaling/OWNERS b/test/e2e/autoscaling/OWNERS index 36021184738..b9e68cd9de4 100644 --- a/test/e2e/autoscaling/OWNERS +++ b/test/e2e/autoscaling/OWNERS @@ -8,3 +8,5 @@ approvers: - jszczepkowski - sig-autoscaling-maintainers - wasylkowski +labels: +- sig/autoscaling \ No newline at end of file diff --git a/test/e2e/cloud/OWNERS b/test/e2e/cloud/OWNERS index 3d2ae59cdde..14c714d275c 100644 --- a/test/e2e/cloud/OWNERS +++ b/test/e2e/cloud/OWNERS @@ -4,12 +4,11 @@ approvers: - andrewsykim - cheftako - mcrute -- d-nishi reviewers: - andrewsykim - cheftako - mcrute +emeritus_approvers: - d-nishi - labels: - sig/cloud-provider diff --git a/test/e2e/common/OWNERS b/test/e2e/common/OWNERS index 7695cc16b12..542db766c92 100644 --- a/test/e2e/common/OWNERS +++ b/test/e2e/common/OWNERS @@ -2,10 +2,11 @@ approvers: - Random-Liu -- dchen1107 - derekwaynecarr - tallclair -- vishh - yujuhong +emeritus_approvers: +- vishh +- dchen1107 reviewers: - sig-node-reviewers diff --git a/test/e2e/instrumentation/OWNERS b/test/e2e/instrumentation/OWNERS index 00b2875c71f..58bbcca63f8 100644 --- a/test/e2e/instrumentation/OWNERS +++ b/test/e2e/instrumentation/OWNERS @@ -2,10 +2,12 @@ approvers: - fabxc -- fgrzadkowski - sig-instrumentation-approvers - x13n - kawych +emeritus_approvers: +- piosz +- fgrzadkowski reviewers: - sig-instrumentation-reviewers labels: diff --git a/test/e2e/kubectl/OWNERS b/test/e2e/kubectl/OWNERS index d3e63eb0d8a..a00fbf29eeb 100644 --- a/test/e2e/kubectl/OWNERS +++ b/test/e2e/kubectl/OWNERS @@ -4,3 +4,5 @@ approvers: - sig-cli-maintainers reviewers: - sig-cli +labels: +- sig/cli diff --git a/test/e2e/lifecycle/OWNERS b/test/e2e/lifecycle/OWNERS index 598596bf175..3e8a644d680 100644 --- a/test/e2e/lifecycle/OWNERS +++ b/test/e2e/lifecycle/OWNERS @@ -4,3 +4,5 @@ approvers: - sig-cluster-lifecycle-maintainers reviewers: - sig-cluster-lifecycle +labels: +- sig/cluster-lifecycle diff --git a/test/e2e/network/OWNERS b/test/e2e/network/OWNERS index 9af008152f7..b60b6256bb9 100644 --- a/test/e2e/network/OWNERS +++ b/test/e2e/network/OWNERS @@ -4,3 +4,5 @@ reviewers: - sig-network-reviewers approvers: - sig-network-approvers +labels: + - sig/network diff --git a/test/e2e/node/OWNERS b/test/e2e/node/OWNERS index 7695cc16b12..81b5708c6fb 100644 --- a/test/e2e/node/OWNERS +++ b/test/e2e/node/OWNERS @@ -2,10 +2,13 @@ approvers: - Random-Liu -- dchen1107 - derekwaynecarr - tallclair -- vishh - yujuhong +emeritus_approvers: +- vishh +- dchen1107 reviewers: - sig-node-reviewers +labels: +- sig/node \ No newline at end of file diff --git a/test/e2e/scheduling/OWNERS b/test/e2e/scheduling/OWNERS index 60d3f73d36d..485eb202d98 100644 --- a/test/e2e/scheduling/OWNERS +++ b/test/e2e/scheduling/OWNERS @@ -4,3 +4,5 @@ approvers: - sig-scheduling-maintainers reviewers: - sig-scheduling +labels: +- sig/scheduling diff --git a/test/e2e/storage/OWNERS b/test/e2e/storage/OWNERS index 84d0482e2df..35e57443578 100644 --- a/test/e2e/storage/OWNERS +++ b/test/e2e/storage/OWNERS @@ -19,3 +19,5 @@ reviewers: - copejon - verult - davidz627 +labels: +- sig/storage \ No newline at end of file diff --git a/test/e2e/windows/OWNERS b/test/e2e/windows/OWNERS index b610aa1aa50..d3244012687 100644 --- a/test/e2e/windows/OWNERS +++ b/test/e2e/windows/OWNERS @@ -1,7 +1,6 @@ # See the OWNERS docs at https://go.k8s.io/owners approvers: -- dchen1107 - michmike - patricklang reviewers: @@ -12,3 +11,7 @@ reviewers: - feiskyer - madhanrm - yujuhong +emeritus_approvers: +- dchen1107 +labels: +- sig/windows diff --git a/test/e2e_node/OWNERS b/test/e2e_node/OWNERS index 8e3400bb6ec..b2b8e0441a4 100644 --- a/test/e2e_node/OWNERS +++ b/test/e2e_node/OWNERS @@ -3,10 +3,11 @@ approvers: - Random-Liu - tallclair -- vishh - derekwaynecarr - yujuhong -- balajismaniam - ConnorDoyle +emeritus_approvers: +- balajismaniam +- vishh reviewers: - sig-node-reviewers diff --git a/test/images/OWNERS b/test/images/OWNERS index 62ecd84a7aa..0f3640c0495 100644 --- a/test/images/OWNERS +++ b/test/images/OWNERS @@ -1,12 +1,11 @@ # See the OWNERS docs at https://go.k8s.io/owners reviewers: - - luxas - mkumatag - - ixdy - listx approvers: - - luxas - mkumatag - - ixdy - listx +emeritus_approvers: + - luxas + - ixdy \ No newline at end of file diff --git a/test/instrumentation/OWNERS b/test/instrumentation/OWNERS index 46c94893ab5..4123240f7bd 100644 --- a/test/instrumentation/OWNERS +++ b/test/instrumentation/OWNERS @@ -5,5 +5,7 @@ approvers: - logicalhan reviewers: - sig-instrumentation-reviewers +emeritus_approvers: +- piosz labels: - sig/instrumentation diff --git a/test/integration/pods/OWNERS b/test/integration/pods/OWNERS index 7695cc16b12..2b75bf74398 100644 --- a/test/integration/pods/OWNERS +++ b/test/integration/pods/OWNERS @@ -2,10 +2,11 @@ approvers: - Random-Liu -- dchen1107 - derekwaynecarr - tallclair -- vishh - yujuhong reviewers: - sig-node-reviewers +emeritus_approvers: +- vishh +- dchen1107 diff --git a/test/integration/statefulset/OWNERS b/test/integration/statefulset/OWNERS deleted file mode 100644 index 0d46fbf12db..00000000000 --- a/test/integration/statefulset/OWNERS +++ /dev/null @@ -1,8 +0,0 @@ -# See the OWNERS docs at https://go.k8s.io/owners - -reviewers: - - barney-s - - crimsonfaith91 -approvers: - - barney-s - - crimsonfaith91 diff --git a/test/kubemark/OWNERS b/test/kubemark/OWNERS index 329e8801dc9..ae065c1e5e4 100644 --- a/test/kubemark/OWNERS +++ b/test/kubemark/OWNERS @@ -1,10 +1,11 @@ # See the OWNERS docs at https://go.k8s.io/owners reviewers: - - gmarek - - shyamjvs - wojtek-t approvers: + - wojtek-t +emeritus_approvers: - gmarek - shyamjvs - - wojtek-t +labels: + - sig/scalability diff --git a/test/typecheck/OWNERS b/test/typecheck/OWNERS index ec2e1481307..f44f602ae8a 100644 --- a/test/typecheck/OWNERS +++ b/test/typecheck/OWNERS @@ -5,4 +5,5 @@ reviewers: - cblecker approvers: - cblecker - - rmmh +emeritus_approvers: + - rmmh \ No newline at end of file