From f81fc8ae1b8e616c63fe9acaa66eafad0168655b Mon Sep 17 00:00:00 2001 From: Kenneth Owens Date: Tue, 16 Apr 2019 11:30:31 -0700 Subject: [PATCH] Updates OWNERS files and OWNERS_ALIASES for SIG Apps to make reviewers and approvers more clear --- OWNERS_ALIASES | 10 ++++++++-- pkg/controller/cronjob/OWNERS | 8 ++------ pkg/controller/daemon/OWNERS | 10 ++-------- pkg/controller/deployment/OWNERS | 10 ++-------- pkg/controller/disruption/OWNERS | 2 ++ pkg/controller/replicaset/OWNERS | 9 ++------- pkg/controller/replication/OWNERS | 9 ++------- pkg/controller/statefulset/OWNERS | 14 +++----------- 8 files changed, 23 insertions(+), 49 deletions(-) diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES index b7a8253c1de..574c4625590 100644 --- a/OWNERS_ALIASES +++ b/OWNERS_ALIASES @@ -223,14 +223,20 @@ aliases: - nicksardo - thockin - rramkumar1 + sig-apps-approvers: + - kow3ns + - janetkuo + - soltysh + - tnozicka + - erictune + - smarterclayton sig-apps-reviewers: - enisoc - erictune - foxish - janetkuo - kow3ns - - lukaszo - - mfojtik + - mortent - smarterclayton - soltysh - tnozicka diff --git a/pkg/controller/cronjob/OWNERS b/pkg/controller/cronjob/OWNERS index 2941b561f02..938e26b5eed 100644 --- a/pkg/controller/cronjob/OWNERS +++ b/pkg/controller/cronjob/OWNERS @@ -1,12 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners approvers: -- erictune -- janetkuo -- soltysh +- sig-apps-approvers reviewers: -- erictune -- janetkuo -- soltysh +- sig-apps-reviewers labels: - sig/apps diff --git a/pkg/controller/daemon/OWNERS b/pkg/controller/daemon/OWNERS index 8ec6466c0dc..938e26b5eed 100644 --- a/pkg/controller/daemon/OWNERS +++ b/pkg/controller/daemon/OWNERS @@ -1,14 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners approvers: -- mikedanese -- janetkuo -- k82cn +- sig-apps-approvers reviewers: -- janetkuo -- lukaszo -- mikedanese -- tnozicka -- k82cn +- sig-apps-reviewers labels: - sig/apps diff --git a/pkg/controller/deployment/OWNERS b/pkg/controller/deployment/OWNERS index cfda9340c32..938e26b5eed 100644 --- a/pkg/controller/deployment/OWNERS +++ b/pkg/controller/deployment/OWNERS @@ -1,14 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners approvers: -- janetkuo -- nikhiljindal -- kargakis -- mfojtik +- sig-apps-approvers reviewers: -- janetkuo -- kargakis -- mfojtik -- tnozicka +- sig-apps-reviewers labels: - sig/apps diff --git a/pkg/controller/disruption/OWNERS b/pkg/controller/disruption/OWNERS index 53e9977950f..938e26b5eed 100644 --- a/pkg/controller/disruption/OWNERS +++ b/pkg/controller/disruption/OWNERS @@ -1,5 +1,7 @@ # See the OWNERS docs at https://go.k8s.io/owners +approvers: +- sig-apps-approvers reviewers: - sig-apps-reviewers labels: diff --git a/pkg/controller/replicaset/OWNERS b/pkg/controller/replicaset/OWNERS index b8444207c7a..938e26b5eed 100644 --- a/pkg/controller/replicaset/OWNERS +++ b/pkg/controller/replicaset/OWNERS @@ -1,13 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners approvers: -- caesarxuchao -- lavalamp -- enisoc +- sig-apps-approvers reviewers: -- caesarxuchao -- lavalamp -- tnozicka -- enisoc +- sig-apps-reviewers labels: - sig/apps diff --git a/pkg/controller/replication/OWNERS b/pkg/controller/replication/OWNERS index b8444207c7a..938e26b5eed 100644 --- a/pkg/controller/replication/OWNERS +++ b/pkg/controller/replication/OWNERS @@ -1,13 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners approvers: -- caesarxuchao -- lavalamp -- enisoc +- sig-apps-approvers reviewers: -- caesarxuchao -- lavalamp -- tnozicka -- enisoc +- sig-apps-reviewers labels: - sig/apps diff --git a/pkg/controller/statefulset/OWNERS b/pkg/controller/statefulset/OWNERS index f0156c7d4c7..39ba14e67aa 100644 --- a/pkg/controller/statefulset/OWNERS +++ b/pkg/controller/statefulset/OWNERS @@ -1,17 +1,9 @@ # See the OWNERS docs at https://go.k8s.io/owners approvers: -- enisoc -- foxish -- janetkuo -- kow3ns -- smarterclayton +- sig-apps-approvers reviewers: -- enisoc -- foxish -- janetkuo -- kow3ns -- smarterclayton -- tnozicka +- sig-apps-reviewers labels: - sig/apps +