Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert OWNERS changes #544

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
17 changes: 17 additions & 0 deletions .github/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
# See the OWNERS docs at https://go.k8s.io/owners

reviewers:
- castrojo
- cblecker
- guineveresaenger
- justaugustus
- mrbobbytables
- nikhita
- parispittman
approvers:
- sig-contribex-approvers
- parispittman
emeritus_approvers:
- Phillels
labels:
- sig/contributor-experience
11 changes: 11 additions & 0 deletions CHANGELOG/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
# See the OWNERS docs at https://go.k8s.io/owners

approvers:
- sig-release-approvers
- release-engineering-approvers
reviewers:
- release-engineering-reviewers

labels:
- sig/release
- area/release-eng
3 changes: 2 additions & 1 deletion DOWNSTREAM_OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

filters:
".*":
# Downstream reviewers, don't have to match those in OWNERS
reviewers:
- deads2k
- sttts
Expand All @@ -10,7 +11,7 @@ filters:
- marun
- tnozicka

# approvers are limited to the team that manages rebases and pays the price for carries that are introduced
# Approvers are limited to the team that manages rebases and pays the price for carries that are introduced
approvers:
- deads2k
- sttts
Expand Down
4 changes: 4 additions & 0 deletions LICENSES/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
# See the OWNERS docs at https://go.k8s.io/owners

approvers:
- dep-approvers
48 changes: 26 additions & 22 deletions OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -3,29 +3,33 @@
filters:
".*":
reviewers:
- deads2k
- sttts
- soltysh
- mfojtik
- marun
- tnozicka

# approvers are limited to the team that manages rebases and pays the price for carries that are introduced
- brendandburns
- dchen1107
- lavalamp
- smarterclayton
- thockin
- liggitt
approvers:
- deads2k
- sttts
- soltysh
- mfojtik
- marun
- tnozicka
- dchen1107
- lavalamp
- smarterclayton
- thockin
- wojtek-t
- liggitt
emeritus_approvers:
- brendandburns
- bgrant0607
- jbeda

"^\\.go.(mod|sum)$":
labels:
- "vendor-update"
"^vendor/.*":
# go.{mod,sum} files relate to go dependencies, and should be reviewed by the
# dep-approvers
"go\\.(mod|sum)$":
required_reviewers:
- kubernetes/dep-approvers
labels:
- "vendor-update"
"^staging/.*":
- area/dependency
# metrics.go files are sig-instrumentation related, and should be tagged
# and reviewed by sig-instrumentation
"metrics\\.go$":
labels:
- "vendor-update"
component: kube-apiserver
- sig/instrumentation