Skip to content

Commit

Permalink
Update k8s-infra-prow groups, add prow OWNERS files
Browse files Browse the repository at this point in the history
Specifically

- k8s-infra-prow-oncall
  - reconcile membership with test-infra-oncall
  - reconcile membership with sig-testing-leads
- k8s-infra-prow-viewers
  - replace scalability-oncall members with group
- prow OWNERS files
  - reviewers: approvers + test-infra-oncall + releng reviewers
  - cluster approvers: sig-testing-leads + ameukam
  - cluster resource approvers: cluster approvers + test-infra-oncall
  - bash approvers: wg-k8s-infra-org-admins

There is prow-related stuff littered in many of the infra/gcp scripts,
so to get things to a point where k8s-infra-prow-oncall members can
approve and run prow-related script changes is going to require more
of a refactor than is relevant to the scope of this PR
  • Loading branch information
spiffxp committed Feb 10, 2021
1 parent 5b2304f commit 31cf984
Show file tree
Hide file tree
Showing 8 changed files with 88 additions and 12 deletions.
14 changes: 14 additions & 0 deletions OWNERS_ALIASES
Original file line number Diff line number Diff line change
Expand Up @@ -241,4 +241,18 @@ aliases:
- saschagrunert # subproject owner / Release Manager / SIG Chair
- tpepper # subproject owner
- xmudrii # Release Manager
# business hours oncall for prow etc (https://go.k8s.io/oncall)
test-infra-oncall:
- chaodaiG
- chases2
- cjwagner
- e-blackwelder
- fejta
- michelle192837
# people with sufficient privileges to run scripts in /infra/gcp
wg-k8s-infra-org-admins:
- cblecker
- dims
- spiffxp
- thockin
## END CUSTOM CONTENT
20 changes: 8 additions & 12 deletions groups/groups.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -651,17 +651,20 @@ groups:
members:
# test-infra-oncall
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
# wg-k8s-infra-oncall
# sig-testing leads
- [email protected]
- [email protected]
- [email protected]
- [email protected]
# wg-k8s-infra members
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]

- email-id: [email protected]
Expand Down Expand Up @@ -691,14 +694,7 @@ groups:
- [email protected]
- [email protected]
- [email protected]
# sig-scalability
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]

- email-id: [email protected]
name: k8s-infra-sig-scalability-oncall
Expand Down
13 changes: 13 additions & 0 deletions infra/gcp/clusters/projects/k8s-infra-prow-build-trusted/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# See the OWNERS docs at https://go.k8s.io/owners

reviewers:
- sig-testing-leads
- release-engineering-reviewers
- test-infra-oncall
approvers:
- sig-testing-leads
- ameukam

labels:
- area/prow
- sig/testing
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# See the OWNERS docs at https://go.k8s.io/owners

reviewers:
- sig-testing-leads
- release-engineering-reviewers
- test-infra-oncall
approvers:
- sig-testing-leads
- test-infra-oncall
- ameukam

labels:
- area/prow
- sig/testing
13 changes: 13 additions & 0 deletions infra/gcp/clusters/projects/k8s-infra-prow-build/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# See the OWNERS docs at https://go.k8s.io/owners

reviewers:
- sig-testing-leads
- release-engineering-reviewers
- test-infra-oncall
approvers:
- sig-testing-leads
- ameukam

labels:
- area/prow
- sig/testing
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# See the OWNERS docs at https://go.k8s.io/owners

reviewers:
- sig-testing-leads
- release-engineering-reviewers
- test-infra-oncall
approvers:
- sig-testing-leads
- test-infra-oncall
- ameukam

labels:
- area/prow
- sig/testing
12 changes: 12 additions & 0 deletions infra/gcp/prow/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
# See the OWNERS docs at https://go.k8s.io/owners

reviewers:
- sig-testing-leads
- release-engineering-reviewers
- test-infra-oncall
approvers:
- wg-k8s-infra-org-admins

labels:
- area/prow
- sig/testing
File renamed without changes.

0 comments on commit 31cf984

Please sign in to comment.