Skip to content

Commit

Permalink
Introduce sig-instrumentation aliases in OWNERS_ALISES and simplify O…
Browse files Browse the repository at this point in the history
…WNERS files
  • Loading branch information
serathius committed Oct 10, 2019
1 parent 2d8b870 commit c601d34
Show file tree
Hide file tree
Showing 13 changed files with 53 additions and 47 deletions.
14 changes: 14 additions & 0 deletions OWNERS_ALIASES
Original file line number Diff line number Diff line change
Expand Up @@ -273,6 +273,20 @@ aliases:
- bskiba
- MaciekPytel
- mwielgus
sig-instrumentation-approvers:
- piosz
- brancz
sig-instrumentation-reviewers:
- kawych
- s-urbaniak
- DirectXMan12
- x13n
- loburm
- huangyuqi
- andyxning
- coffeepac
- monotek
- logicalhan

api-approvers:
- erictune
Expand Down
7 changes: 4 additions & 3 deletions cluster/addons/cluster-monitoring/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@

approvers:
- kawych
- piosz
- sig-instrumentation-approvers
reviewers:
- kawych
- piosz
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
6 changes: 2 additions & 4 deletions cluster/addons/fluentd-elasticsearch/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,8 @@
approvers:
- coffeepac
- monotek
- piosz
- sig-instrumentation-approvers
reviewers:
- coffeepac
- monotek
- piosz
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
6 changes: 3 additions & 3 deletions cluster/addons/fluentd-gcp/OWNERS
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# See the OWNERS docs at https://go.k8s.io/owners

approvers:
- piosz
- sig-instrumentation-approvers
- x13n
reviewers:
- piosz
- x13n
- sig-instrumentation-reviewers
labels:
- sig/gcp
- sig/instrumentation
8 changes: 4 additions & 4 deletions cluster/addons/metadata-agent/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@

approvers:
- kawych
- piosz
- sig-instrumentation-approvers
- x13n
reviewers:
- kawych
- piosz
- x13n
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
7 changes: 4 additions & 3 deletions cluster/addons/metrics-server/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@

approvers:
- kawych
- piosz
- sig-instrumentation-approvers
reviewers:
- kawych
- piosz
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
7 changes: 2 additions & 5 deletions staging/src/k8s.io/component-base/metrics/OWNERS
Original file line number Diff line number Diff line change
@@ -1,12 +1,9 @@
# See the OWNERS docs at https://go.k8s.io/owners

approvers:
- piosz
- brancz
- sig-instrumentation-approvers
- logicalhan
reviewers:
- piosz
- brancz
- logicalhan
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
Original file line number Diff line number Diff line change
@@ -1,12 +1,9 @@
# See the OWNERS docs at https://go.k8s.io/owners

approvers:
- piosz
- brancz
- sig-instrumentation-approvers
- logicalhan
reviewers:
- piosz
- brancz
- logicalhan
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
4 changes: 3 additions & 1 deletion staging/src/k8s.io/metrics/OWNERS
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
# See the OWNERS docs at https://go.k8s.io/owners

approvers:
- piosz
- sig-instrumentation-approvers
labels:
- sig/instrumentation
6 changes: 4 additions & 2 deletions test/e2e/instrumentation/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,10 @@
approvers:
- fabxc
- fgrzadkowski
- piosz
- sig-instrumentation-approvers
- x13n
- kawych
reviewers:
- sig-instrumentation-pr-reviews
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
14 changes: 7 additions & 7 deletions test/e2e/instrumentation/logging/OWNERS
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# See the OWNERS docs at https://go.k8s.io/owners

reviewers:
- coffeepac
- monotek
- piosz
approvers:
- coffeepac
- monotek
- piosz
- coffeepac
- monotek
- sig-instrumentation-approvers
reviewers:
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
7 changes: 2 additions & 5 deletions test/instrumentation/OWNERS
Original file line number Diff line number Diff line change
@@ -1,12 +1,9 @@
# See the OWNERS docs at https://go.k8s.io/owners

approvers:
- piosz
- brancz
- sig-instrumentation-approvers
- logicalhan
reviewers:
- piosz
- brancz
- logicalhan
- sig-instrumentation-reviewers
labels:
- sig/instrumentation
7 changes: 2 additions & 5 deletions test/instrumentation/testdata/OWNERS
Original file line number Diff line number Diff line change
@@ -1,11 +1,8 @@
# See the OWNERS docs at https://go.k8s.io/owners

approvers:
- piosz
- brancz
- sig-instrumentation-approvers
reviewers:
- piosz
- brancz
- logicalhan
- sig-instrumentation-reviewers
labels:
- sig/instrumentation

0 comments on commit c601d34

Please sign in to comment.