Skip to content

Commit

Permalink
owners, user-guide: set up owners from sigs (#819)
Browse files Browse the repository at this point in the history
Since we have the sigs formed for the several topics, let's establish
ownership of the sections by each respective sig.

Also we remove some users from the reviewers section who haven't shown
review activity for the last six months - this way we avoid assigning
them to the reviews and getting reviews stalled.

See https://kubevirt.devstats.cncf.io/d/46/pr-reviews-by-contributor?orgId=1&var-period=d7&var-repo_name=kubevirt%2Fuser-guide&var-reviewers=%22davidvossel%22&var-reviewers=%22rmohr%22&var-reviewers=%22stu-gott%22

Signed-off-by: Daniel Hiller <[email protected]>
  • Loading branch information
dhiller authored Jul 19, 2024
1 parent 018ab56 commit 678739d
Show file tree
Hide file tree
Showing 7 changed files with 186 additions and 29 deletions.
36 changes: 7 additions & 29 deletions OWNERS
Original file line number Diff line number Diff line change
@@ -1,37 +1,15 @@
filters:
".*":
reviewers:
- davidvossel
- vladikr
- rmohr
- stu-gott
- fabiand
- AlonaKaplan
- dhiller
- jean-edouard
- mhenriks
- phoracek
- cwilkers
- jobbler
- aburdenthehand
- reviewers
approvers:
- davidvossel
- vladikr
- rmohr
- stu-gott
- fabiand
- AlonaKaplan
- dhiller
- jean-edouard
- mhenriks
- phoracek
- cwilkers
- approvers
emeritus_approvers:
- ashleyschuett
- codificat
- hroyrh
- danielBelenky
- mazzystr
- ashleyschuett
- codificat
- hroyrh
- danielBelenky
- mazzystr

"^docs/.*":
labels:
Expand Down
139 changes: 139 additions & 0 deletions OWNERS_ALIASES
Original file line number Diff line number Diff line change
@@ -0,0 +1,139 @@
aliases:
reviewers:
- vladikr
- fabiand
- AlonaKaplan
- dhiller
- jean-edouard
- mhenriks
- phoracek
- cwilkers
- jobbler
- aburdenthehand
approvers:
- davidvossel
- vladikr
- rmohr
- stu-gott
- fabiand
- AlonaKaplan
- dhiller
- jean-edouard
- mhenriks
- phoracek
- cwilkers

#
# SIG Test
#
sig-test-reviewers:
- kbidarkar
sig-test-approvers:
- kbidarkar
- phoracek
- enp0s3
- xpivarc
- acardace
- dhiller
- AlonaKaplan

#
# SIG Network
# Owns anything related to networking.
#
sig-network-reviewers:
- AlonaKaplan
- EdDev
- RamLavi
- ormergi
sig-network-approvers:
- AlonaKaplan
- EdDev

#
# SIG Scale
# Owns to keep kubevirt's scalability comparable to Kubernetes'.
#
sig-scale-approvers:
- rthallisey
sig-scale-reviewers:
- rthallisey

#
# SIG Storage
# Owns anything related to storage.
#
sig-storage-approvers:
- mhenriks
- alicefr
sig-storage-reviewers:
- awels
- akalenyu
- ShellyKa13

#
# SIG API
# Owns the API including API life-cycle, deprecation,
# and backwards compatibility.
#
sig-api-approvers: []
sig-api-reviewers: []

#
# SIG Compute
# Owns everything which is taking place on a node, for example
# (but not limited to) groups, SELinux, node labels, …
# And everything on the cluster level such as RBAC, controller, …
#
sig-compute-approvers:
- jean-edouard
- iholder101
sig-compute-reviewers:
- victortoso

#
# SIG Observability
# Owns the responsibility to keep kubevirt observable by i.e.
# having mertics, alters, and runbooks.
#
sig-observability-approvers:
- sradco
- machadovilaca
sig-observability-reviewers:
- machadovilaca
- avlitman
- assafad

#
# SIG Release
# Owns the release process, including the schedule, and tools.
#
sig-release-approvers:
- acardace
- fossedihelm
- xpivarc
sig-release-reviewers:
- acardace
- fossedihelm
- xpivarc

#
# SIG Buildsystem
# Owns bazel, and ensures that kubevirt can be build.
#
sig-buildsystem-approvers:
- brianmcarey
- dhiller
- xpivarc
sig-buildsystem-reviewers:
- brianmcarey
- enp0s3
- xpivarc

#
# SIG Architecture
# Owns the overall architecture, and supporting the growth, health,
# openess of KubeVirt.
#
sig-architecture-approvers: []
sig-architecture-reviewers: []
8 changes: 8 additions & 0 deletions docs/cluster_admin/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
filters:
".*":
reviewers:
- sig-compute-reviewers
approvers:
- sig-compute-approvers
emeritus_approvers:
- emeritus_approvers
8 changes: 8 additions & 0 deletions docs/compute/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
filters:
".*":
reviewers:
- sig-compute-reviewers
approvers:
- sig-compute-approvers
emeritus_approvers:
- emeritus_approvers
8 changes: 8 additions & 0 deletions docs/debug_virt_stack/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
filters:
".*":
reviewers:
- sig-compute-reviewers
approvers:
- sig-compute-approvers
emeritus_approvers:
- emeritus_approvers
8 changes: 8 additions & 0 deletions docs/network/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
filters:
".*":
reviewers:
- sig-network-reviewers
approvers:
- sig-network-approvers
emeritus_approvers:
- emeritus_approvers
8 changes: 8 additions & 0 deletions docs/storage/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
filters:
".*":
reviewers:
- sig-storage-reviewers
approvers:
- sig-storage-approvers
emeritus_approvers:
- emeritus_approvers

0 comments on commit 678739d

Please sign in to comment.