Skip to content

Commit 71e9a16

Browse files
committed
Clean up OWNERS file
This cleans up the owners file, making sure that only active members will be assigned and bringing it in line with the latest docs on the OWNERS file from the main k/community repo.
1 parent cb6ce28 commit 71e9a16

File tree

3 files changed

+25
-13
lines changed

3 files changed

+25
-13
lines changed

OWNERS

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,9 @@
1-
# See the OWNERS docs: https://git.k8s.io/community/contributors/devel/owners.md
2-
owners:
3-
- directxman12
4-
- droot
5-
- pwittrock
1+
# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md
2+
63
approvers:
74
- controller-runtime-admins
8-
- controller-runtime-maintainers
5+
- controller-runtime-approvers
96
reviewers:
107
- controller-runtime-admins
11-
- controller-runtime-maintainers
128
- controller-runtime-reviewers
9+
- controller-runtime-approvers

OWNERS_ALIASES

Lines changed: 20 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,35 @@
1-
# See the OWNERS docs: https://git.k8s.io/community/contributors/devel/owners.md
1+
# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md
22

33
aliases:
4+
# active folks who can be contacted to perform admin-related
5+
# tasks on the repo, or otherwise approve any PRS.
46
controller-runtime-admins:
57
- directxman12
68
- droot
7-
- pwittrock
8-
controller-runtime-maintainers: []
9+
- mengqiy
10+
11+
# non-admin folks who can approve any PRs in the repo
12+
controller-runtime-approvers: []
13+
14+
# folks who can review and LGTM any PRs in the repo (doesn't
15+
# include approvers & admins -- those count too via the OWNERS
16+
# file)
917
controller-runtime-reviewers:
1018
- alvaroaleman
1119
- shawn-hurley
1220
- gerred
1321
- joelanford
1422
- alenkacz
1523
- vincepri
16-
testing-integration-admins:
24+
25+
# folks to can approve things in the directly-ported
26+
# testing_frameworks portions of the codebase
27+
testing-integration-approvers:
1728
- apelisse
1829
- hoegaarden
1930
- totherme
31+
32+
# folks who may have context on ancient history,
33+
# but are no longer directly involved
34+
controller-runtime-emeritus-maintainers:
35+
- pwittrock

pkg/internal/testing/OWNERS

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
# See the OWNERS docs: https://git.k8s.io/community/contributors/devel/owners.md
22

33
approvers:
4-
- controller-runtime-admins
5-
- testing-integration-admins
4+
- testing-integration-approvers

0 commit comments

Comments
 (0)