From 7c50cd762a7f69d6bef7774a67567fbe0bbfce2e Mon Sep 17 00:00:00 2001 From: Aldo Culquicondor Date: Mon, 17 May 2021 15:34:55 +0000 Subject: [PATCH] Use aliases in sig apps and scheduling APIs OWNERS Add alculquicondor to sig-scheduling-api-reviewers and sig-apps-api-reviewers --- OWNERS_ALIASES | 6 +++--- pkg/apis/apps/OWNERS | 16 +++------------- pkg/apis/batch/OWNERS | 15 +++------------ pkg/apis/policy/OWNERS | 1 + pkg/controller/apis/config/OWNERS | 3 +++ pkg/scheduler/apis/config/OWNERS | 15 ++++++++------- staging/src/k8s.io/api/apps/OWNERS | 18 +++++------------- staging/src/k8s.io/api/batch/OWNERS | 17 +++++------------ .../src/k8s.io/kube-scheduler/config/OWNERS | 3 ++- 9 files changed, 33 insertions(+), 61 deletions(-) diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES index 76c99a77c4c..0992a924ed9 100644 --- a/OWNERS_ALIASES +++ b/OWNERS_ALIASES @@ -452,6 +452,7 @@ aliases: - janetkuo - kow3ns - soltysh + - alculquicondor sig-auth-api-reviewers: - enj @@ -500,9 +501,8 @@ aliases: - mm4tt - wojtek-t - # sig-scheduling-api-reviewers: - # - - # - + sig-scheduling-api-reviewers: + - alculquicondor sig-storage-api-reviewers: - deads2k diff --git a/pkg/apis/apps/OWNERS b/pkg/apis/apps/OWNERS index 11c1b1a4fbb..ec350aa4ff8 100644 --- a/pkg/apis/apps/OWNERS +++ b/pkg/apis/apps/OWNERS @@ -1,18 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners +# approval on api packages bubbles to api-approvers reviewers: -- thockin -- lavalamp -- smarterclayton -- deads2k -- caesarxuchao -- pmorie -- sttts -- saad-ali -- ncdc -- dims -- errordeveloper -- m1093782566 -- kevin-wangzefeng +- sig-apps-api-reviewers +- sig-apps-api-approvers labels: - sig/apps diff --git a/pkg/apis/batch/OWNERS b/pkg/apis/batch/OWNERS index 604da6c64a9..ec350aa4ff8 100644 --- a/pkg/apis/batch/OWNERS +++ b/pkg/apis/batch/OWNERS @@ -1,17 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners +# approval on api packages bubbles to api-approvers reviewers: -- thockin -- lavalamp -- smarterclayton -- wojtek-t -- deads2k -- caesarxuchao -- sttts -- saad-ali -- ncdc -- soltysh -- dims -- errordeveloper +- sig-apps-api-reviewers +- sig-apps-api-approvers labels: - sig/apps diff --git a/pkg/apis/policy/OWNERS b/pkg/apis/policy/OWNERS index 31a0a7358d6..09f9d9f7c32 100644 --- a/pkg/apis/policy/OWNERS +++ b/pkg/apis/policy/OWNERS @@ -2,6 +2,7 @@ # approval on api packages bubbles to api-approvers reviewers: +- sig-apps-api-reviewers - sig-apps-api-approvers - sig-auth-policy-approvers - sig-auth-policy-reviewers diff --git a/pkg/controller/apis/config/OWNERS b/pkg/controller/apis/config/OWNERS index 5381816e41e..4c7c9d9b532 100644 --- a/pkg/controller/apis/config/OWNERS +++ b/pkg/controller/apis/config/OWNERS @@ -1,5 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners +# Disable inheritance as this is an api owners file +options: + no_parent_owners: true approvers: - api-approvers - deads2k diff --git a/pkg/scheduler/apis/config/OWNERS b/pkg/scheduler/apis/config/OWNERS index 17b616c71cc..26561c6513f 100644 --- a/pkg/scheduler/apis/config/OWNERS +++ b/pkg/scheduler/apis/config/OWNERS @@ -1,13 +1,14 @@ # See the OWNERS docs at https://go.k8s.io/owners +# Disable inheritance as this is an api owners file +options: + no_parent_owners: true approvers: - api-approvers -- sig-scheduling-maintainers -- sttts -- luxas reviewers: -- sig-scheduling - api-reviewers -- dixudx -- luxas -- sttts +- sig-scheduling-api-reviewers +- sig-scheduling-api-approvers +labels: +- kind/api-change +- sig/scheduling diff --git a/staging/src/k8s.io/api/apps/OWNERS b/staging/src/k8s.io/api/apps/OWNERS index a50ce7de010..ec350aa4ff8 100644 --- a/staging/src/k8s.io/api/apps/OWNERS +++ b/staging/src/k8s.io/api/apps/OWNERS @@ -1,16 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners +# approval on api packages bubbles to api-approvers reviewers: -- thockin -- lavalamp -- smarterclayton -- deads2k -- caesarxuchao -- pmorie -- sttts -- saad-ali -- ncdc -- dims -- errordeveloper -- m1093782566 -- kevin-wangzefeng +- sig-apps-api-reviewers +- sig-apps-api-approvers +labels: +- sig/apps diff --git a/staging/src/k8s.io/api/batch/OWNERS b/staging/src/k8s.io/api/batch/OWNERS index 9ccc6dc0d46..ec350aa4ff8 100644 --- a/staging/src/k8s.io/api/batch/OWNERS +++ b/staging/src/k8s.io/api/batch/OWNERS @@ -1,15 +1,8 @@ # See the OWNERS docs at https://go.k8s.io/owners +# approval on api packages bubbles to api-approvers reviewers: -- thockin -- lavalamp -- smarterclayton -- wojtek-t -- deads2k -- caesarxuchao -- sttts -- saad-ali -- ncdc -- soltysh -- dims -- errordeveloper +- sig-apps-api-reviewers +- sig-apps-api-approvers +labels: +- sig/apps diff --git a/staging/src/k8s.io/kube-scheduler/config/OWNERS b/staging/src/k8s.io/kube-scheduler/config/OWNERS index 9584e0e8313..c4329fea4ee 100644 --- a/staging/src/k8s.io/kube-scheduler/config/OWNERS +++ b/staging/src/k8s.io/kube-scheduler/config/OWNERS @@ -7,6 +7,7 @@ approvers: - api-approvers reviewers: - api-reviewers -- sig-scheduling +- sig-scheduling-api-reviewers +- sig-scheduling-api-approvers labels: - kind/api-change