Move root approvers to subdirs

dprotaso · Oct 10, 2022 · dd09106 · dd09106
1 parent b8b86bb
commit dd09106
Show file tree

Hide file tree

Showing 9 changed files with 55 additions and 12 deletions.
diff --git a/OWNERS b/OWNERS
@@ -2,24 +2,18 @@
 
 filters:
   ".*":
-    reviewers:
+    reviewers: []
+    approvers: []
+    emeritus_approvers:
+      - bgrant0607
+      - brendandburns
       - dchen1107
+      - jbeda
       - lavalamp
-      - smarterclayton
-      - thockin
-      - wojtek-t
       - liggitt
-    approvers:
-      - dchen1107
-      - lavalamp
       - smarterclayton
       - thockin
       - wojtek-t
-      - liggitt
-    emeritus_approvers:
-      - brendandburns
-      - bgrant0607
-      - jbeda
   # go.{mod,sum} files relate to go dependencies, and should be reviewed by the
   # dep-approvers
   "go\\.(mod|sum)$":

diff --git a/build/OWNERS b/build/OWNERS
@@ -11,10 +11,12 @@ reviewers:
   - jeremyrickard # SIG Technical Lead / RelEng subproject owner
   #- justaugustus # SIG Chair / RelEng subproject owner / Release Manager - approvals only
   - lavalamp
+  - liggitt
   - palnabarun # Release Manager
   - puerco # SIG Technical Lead / RelEng subproject owner / Release Manager
   - saschagrunert # SIG Chair / RelEng subproject owner / Release Manager
   - spiffxp
+  - thockin
   - Verolop # Release Manager
   - xmudrii # Release Manager
 approvers:
@@ -23,8 +25,10 @@ approvers:
   - dims
   - justaugustus
   - lavalamp
+  - liggitt
   - mikedanese
   - spiffxp
+  - thockin
 emeritus_approvers:
   - fejta
   - jbeda

diff --git a/cluster/OWNERS b/cluster/OWNERS
@@ -8,14 +8,18 @@ reviewers:
   - cheftako
   - dims
   - justaugustus
+  - liggitt
   - mikedanese
   - spiffxp
+  - wojtek-t
 approvers:
   - bentheelder
   - cheftako
   - dims
+  - liggitt
   - mikedanese
   - spiffxp
+  - wojtek-t
 emeritus_approvers:
   - eparis
   - roberthbailey # 2019-03-08

diff --git a/cmd/OWNERS b/cmd/OWNERS
@@ -7,11 +7,17 @@ reviewers:
   - dchen1107
   - dims
   - lavalamp
+  - liggitt
   - mikedanese
+  - smarterclayton
   - thockin
+  - wojtek-t
 approvers:
   - dchen1107
   - dims
   - lavalamp
+  - liggitt
   - mikedanese
+  - smarterclayton
   - thockin
+  - wojtek-t
diff --git a/hack/OWNERS b/hack/OWNERS
@@ -6,16 +6,22 @@ options:
 reviewers:
   - bentheelder
   - cblecker
+  - dchen1107
   - hwdef
   - justaugustus
   - lavalamp
+  - liggitt
   - mikedanese
   - SataQiu
+  - smarterclayton
   - spiffxp
   - sttts
+  - thockin
+  - wojtek-t
 approvers:
   - bentheelder
   - cblecker
+  - dchen1107
   - deads2k
   - dims # for local-up-cluster related files
   - enj # for sig-auth related stuff like cert testdata
@@ -24,9 +30,12 @@ approvers:
   - mikedanese
   - pwittrock
   - SataQiu
+  - smarterclayton
   - soltysh # for sig-cli related stuff
   - spiffxp
   - sttts
+  - thockin
+  - wojtek-t
 emeritus_approvers:
   - eparis
   - fejta

diff --git a/pkg/OWNERS b/pkg/OWNERS
@@ -7,13 +7,17 @@ reviewers:
   - dchen1107
   - dims
   - lavalamp
+  - liggitt
   - smarterclayton
   - thockin
+  - wojtek-t
 approvers:
   - dchen1107
   - dims
   - lavalamp
+  - liggitt
   - smarterclayton
   - thockin
+  - wojtek-t
 emeritus_approvers:
   - brendandburns
diff --git a/plugin/OWNERS b/plugin/OWNERS
@@ -7,12 +7,18 @@ reviewers:
   - dchen1107
   - dims
   - lavalamp
+  - liggitt
+  - smarterclayton
   - thockin
+  - wojtek-t
 approvers:
   - dchen1107
   - dims
   - lavalamp
+  - liggitt
+  - smarterclayton
   - thockin
+  - wojtek-t
 emeritus_approvers:
   - davidopp
   - brendandburns
diff --git a/staging/OWNERS b/staging/OWNERS
@@ -4,16 +4,22 @@ options:
   # make root approval non-recursive
   no_parent_owners: true
 approvers:
+  - dchen1107
   - dims
   - lavalamp
+  - liggitt
   - smarterclayton
+  - thockin
+  - wojtek-t
 reviewers:
   - caesarxuchao
+  - dchen1107
   - deads2k
   - dims
   - lavalamp
   - liggitt
   - mikedanese
   - smarterclayton
   - sttts
+  - thockin
   - wojtek-t
diff --git a/test/OWNERS b/test/OWNERS
@@ -5,20 +5,28 @@ options:
   no_parent_owners: true
 reviewers:
   - aojea
+  - dchen1107
   - johnSchnake
+  - lavalamp
+  - liggitt
   - neolit123
   - SataQiu
+  - smarterclayton
+  - thockin
+  - wojtek-t
 approvers:
   - andrewsykim
   - aojea
   - BenTheElder
   - bowei # for test/e2e/{dns*,network}.go
   - caseydavenport # for test/e2e/{dns*,network}.go
   - cblecker
+  - dchen1107
   - deads2k
   - dims
   - enj
   - janetkuo
+  - lavalamp
   - liggitt
   - mikedanese
   - MrHohn
@@ -32,6 +40,8 @@ approvers:
   - smarterclayton
   - soltysh
   - sttts
+  - thockin
+  - wojtek-t
 emeritus_approvers:
   - eparis
   - foxish