Define fine-grained owners for the various subcomponents of OLMv1.

joelanford · joelanford · commit ac44a3c40577 · 2025-07-21T23:11:43.000-04:00
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.
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,44 @@
-
 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
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/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/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`