🌱 Define fine-grained owners for the various subcomponents of OLMv1. (#2113)

joelanford · web-flow · commit 0206ad4b4b18 · 2025-07-28T15:38:10.000Z
* Define fine-grained owners for the various subcomponents of OLMv1. The goal is to reduce the possibility of the bystander effect (see https://en.wikipedia.org/wiki/Bystander_effect), and give maintainers more accountability and ownership of the areas in which they are experts. This will also help contributors more quickly identify those experts and get the necessary reviews for their work to merge. * add OWNERS and alias for docs/draft Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * api-approvers for generated CRDs Signed-off-by: Joe Lanford <joe.lanford@gmail.com> --------- Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
diff --git a/.bingo/.gitignore b/.bingo/.gitignore
@@ -9,5 +9,6 @@
 !README.md
 !Variables.mk
 !variables.env
+!OWNERS
 
 *tmp.mod
diff --git a/.bingo/OWNERS b/.bingo/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - ci-approvers
diff --git a/.github/OWNERS b/.github/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - ci-approvers
diff --git a/OWNERS b/OWNERS
@@ -1,5 +1,4 @@
 approvers:
-   - operator-controller-approvers
+   - olmv1-approvers
 reviewers:
-   - operator-controller-approvers
-   - operator-controller-reviewers
+   - olmv1-reviewers
diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES
@@ -1,25 +1,49 @@
-
 aliases:
-   # contributors who can approve any PRs in the repo
-   operator-controller-approvers:
-   - camilamacedo86
-   - grokspawn
+   olmv1-approvers:
    - joelanford
    - kevinrizza
    - perdasilva
-   - thetechnick
    - tmshort
 
-   # contributors who can review/lgtm any PRs in the repo
-   operator-controller-reviewers:
+   olmv1-reviewers:
    - anik120
    - ankitathomas
    - bentito
+   - camilamacedo86
    - dtfranz
-   - gallettilance
-   - gavinmbell
-   - LalatenduMohanty
+   - grokspawn
+   - joelanford
    - oceanc80
    - OchiengEd
+   - perdasilva
    - rashmigottipati
+   - thetechnick
+   - tmshort
    - trgeiger
+
+   api-approvers:
+   - grokspawn
+   - thetechnick
+
+   catalogd-approvers:
+   - grokspawn
+
+   operator-controller-approvers:
+   - thetechnick
+
+   cmd-approvers:
+   - grokspawn
+
+   manifest-approvers:
+   - camilamacedo86
+
+   ci-approvers:
+   - camilamacedo86
+
+   docs-approvers:
+   - michaelryanpeter
+
+   docs-draft-approvers:
+   - camilamacedo86
+   - grokspawn
+   - thetechnick
diff --git a/api/OWNERS b/api/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - api-approvers
diff --git a/cmd/OWNERS b/cmd/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - cmd-approvers
diff --git a/config/OWNERS b/config/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - manifest-approvers
diff --git a/config/base/catalogd/crd/OWNERS b/config/base/catalogd/crd/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - api-approvers
diff --git a/config/base/operator-controller/crd/OWNERS b/config/base/operator-controller/crd/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - api-approvers
diff --git a/docs/OWNERS b/docs/OWNERS
@@ -1,23 +1,2 @@
 approvers:
-   # contributors who can approve any docs PRs in the repo
-   - michaelryanpeter
-reviewers:
-   # contributors who can review/lgtm any docs PRs in the repo
-   - anik120
-   - ankitathomas
-   - bentito
-   - camilamacedo86
-   - dtfranz
-   - gallettilance
-   - gavinmbell
-   - grokspawn
-   - joelanford
-   - kevinrizza
-   - LalatenduMohanty
-   - oceanc80
-   - OchiengEd
-   - perdasilva
-   - rashmigottipati
-   - thetechnick
-   - tmshort
-   - trgeiger
+   - docs-approvers
diff --git a/docs/draft/OWNERS b/docs/draft/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+   - docs-draft-approvers
diff --git a/hack/OWNERS b/hack/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - ci-approvers
diff --git a/internal/catalogd/OWNERS b/internal/catalogd/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - catalogd-approvers
diff --git a/internal/operator-controller/OWNERS b/internal/operator-controller/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - operator-controller-approvers
diff --git a/manifests/OWNERS b/manifests/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - manifest-approvers
diff --git a/scripts/OWNERS b/scripts/OWNERS
@@ -0,0 +1,2 @@
+approvers:
+  - manifest-approvers

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+approvers:`
	`2`	`+ - manifest-approvers`