Skip to content

Commit

Permalink
Owners: Promote Gacko to ingress-nginx-maintainers & `ingress-nginx…
Browse files Browse the repository at this point in the history
…-reviewers`. (#11165)

* Owners: Sort `ingress-nginx-maintainers` & `ingress-nginx-reviewers`.

* Owners: Update URL in aliases.

* Images: Remove owners as it's identical to global owners.

* Images: Remove global owners from `kube-webhook-certgen` owners.

* Owners: Remove members from aliases covered by other aliases.

ingress-nginx-helm-maintainers:
- cpanato: Covered by ingress-nginx-maintainers
- strongjz: Covered by ingress-nginx-maintainers

ingress-nginx-helm-reviewers:
- cpanato: Covered by ingress-nginx-reviewers
- strongjz: Covered by ingress-nginx-reviewers

ingress-nginx-docs-maintainers:
- tao12345666333: Covered by ingress-nginx-maintainers

* Owners: Promote myself to `ingress-nginx-maintainers` & `ingress-nginx-reviewers`.
  • Loading branch information
Gacko committed Apr 4, 2024
1 parent a542bb9 commit bf3fa53
Show file tree
Hide file tree
Showing 7 changed files with 15 additions and 35 deletions.
4 changes: 2 additions & 2 deletions OWNERS
@@ -1,4 +1,4 @@
# See the OWNERS docs: https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md
# See the OWNERS docs: https://www.kubernetes.dev/docs/guide/owners

approvers:
- ingress-nginx-maintainers
Expand All @@ -7,6 +7,6 @@ reviewers:
- ingress-nginx-reviewers

emeritus_approvers:
- aledbf # 2020-04-02
- aledbf # 2020-04-02
- bowei # 2022-10-12
- ElvinEfendi # 2023-04-23
19 changes: 7 additions & 12 deletions OWNERS_ALIASES
@@ -1,4 +1,4 @@
# See the OWNERS docs: https://git.k8s.io/community/docs/devel/owners.md
# See the OWNERS docs: https://www.kubernetes.dev/docs/guide/owners

aliases:
sig-network-leads:
Expand All @@ -11,34 +11,29 @@ aliases:
- strongjz

ingress-nginx-maintainers:
- rikatz
- strongjz
- cpanato
- Gacko
- puerco
- rikatz
- strongjz
- tao12345666333

ingress-nginx-reviewers:
- cpanato
- Gacko
- puerco
- rikatz
- strongjz
- puerco
- cpanato
- tao12345666333

ingress-nginx-helm-maintainers:
- cpanato
- Gacko
- strongjz
- ubergesundheit

ingress-nginx-helm-reviewers:
- cpanato
- Gacko
- strongjz
- ubergesundheit

ingress-nginx-docs-maintainers:
- longwuyuan
- tao12345666333

ingress-nginx-kube-webhook-certgen-reviewers:
- invidian
2 changes: 1 addition & 1 deletion charts/ingress-nginx/OWNERS
@@ -1,4 +1,4 @@
# See the OWNERS docs: https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md
# See the OWNERS docs: https://www.kubernetes.dev/docs/guide/owners

approvers:
- ingress-nginx-helm-maintainers
Expand Down
4 changes: 2 additions & 2 deletions docs/OWNERS
@@ -1,7 +1,7 @@
# See the OWNERS docs: https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md
# See the OWNERS docs: https://www.kubernetes.dev/docs/guide/owners

approvers:
- ingress-nginx-docs-maintainers

labels:
- area/docs
- area/docs
9 changes: 0 additions & 9 deletions images/OWNERS

This file was deleted.

8 changes: 1 addition & 7 deletions images/kube-webhook-certgen/OWNERS
@@ -1,10 +1,4 @@
# See the OWNERS docs: https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md

approvers:
- ingress-nginx-admins
- ingress-nginx-maintainers
# See the OWNERS docs: https://www.kubernetes.dev/docs/guide/owners

reviewers:
- ingress-nginx-admins
- ingress-nginx-maintainers
- ingress-nginx-kube-webhook-certgen-reviewers
4 changes: 2 additions & 2 deletions rootfs/etc/nginx/lua/OWNERS
@@ -1,4 +1,4 @@
# See the OWNERS docs: https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md
# See the OWNERS docs: https://www.kubernetes.dev/docs/guide/owners

labels:
- area/lua
- area/lua

0 comments on commit bf3fa53

Please sign in to comment.