Skip to content

Commit dcb68b8

Browse files
author
Linus Arver
committed
OWNERS: remove chaodaiG from reviewers/approvers
Chao no longer works on Prow or test-infra-related duties (it has been a few months now). This change was (mostly) created with: $ for f in $(git ls-files | grep OWNERS) do sed -i '/chaodaiG/d' $f done For horologium, this resulted in an invalid YAML because Chao was the only reviewer/approver in that OWNERS file. So there cjwagner was added as a replacement. Where Chao was an approver, he was moved to `emeritus_approvers` per the Kubernetes Contributors docs guidelines [1]. [1] https://www.kubernetes.dev/docs/guide/owners/#emeritus
1 parent 215a371 commit dcb68b8

File tree

18 files changed

+26
-35
lines changed

18 files changed

+26
-35
lines changed

‎OWNERS

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ reviewers:
55
- BenTheElder # lead
66
- spiffxp # chair
77
- stevekuznetsov # chair
8-
- chaodaiG # oncall
98
- cjwagner # oncall, lead
109
- mpherman2 # oncall
1110
approvers:

‎OWNERS_ALIASES

-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
aliases:
22
# business hours oncall for prow etc (https://go.k8s.io/oncall)
33
test-infra-oncall:
4-
- chaodaiG
54
- chases2
65
- cjwagner
76
- listx

‎prow/cmd/crier/OWNERS

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,10 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22
reviewers:
3-
- chaodaiG
43
- cjwagner
54
approvers:
6-
- chaodaiG
75
- cjwagner
86
emeritus_approvers:
7+
- chaodaiG
98
- fejta
109
- krzyzacy
1110
labels:

‎prow/cmd/deck/OWNERS

+2-3
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,14 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22
reviewers:
3-
- chaodaiG
43
- michelle192837
54
- mpherman2
65
- smg247
76
approvers:
8-
- chaodaiG
97
- michelle192837
108
- mpherman2
119
- smg247
1210
emeritus_approvers:
13-
- Katharine
11+
- chaodaiG
12+
- Katharine
1413
labels:
1514
- area/prow/deck

‎prow/cmd/generic-autobumper/OWNERS

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- mpherman2
65
approvers:
7-
- chaodaiG
86
- mpherman2
7+
emeritus_approvers:
8+
- chaodaiG
99
labels:
1010
- area/prow/hook

‎prow/cmd/gerrit/OWNERS

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- chases2
65
approvers:
7-
- chaodaiG
86
- chases2
97
emeritus_approvers:
108
- amwat
9+
- chaodaiG
1110
- fejta
1211
- krzyzacy
1312
labels:

‎prow/cmd/horologium/OWNERS

+3-2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
4+
- cjwagner
55
approvers:
6-
- chaodaiG
6+
- cjwagner
77
emeritus_approvers:
8+
- chaodaiG
89
- fejta
910
labels:
1011
- area/prow/horologium

‎prow/cmd/prow-controller-manager/OWNERS

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- cjwagner
65
approvers:
7-
- chaodaiG
86
- cjwagner
7+
emeritus_approvers:
8+
- chaodaiG
99
labels:
1010
- area/prow/plank

‎prow/cmd/sub/OWNERS

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,13 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
approvers:
4-
- chaodaiG
54
- cjwagner
65
- listx
76
reviewers:
8-
- chaodaiG
97
- cjwagner
108
- listx
119
emeritus_approvers:
10+
- chaodaiG
1211
- sebastienvas
1312
labels:
1413
- area/prow/pubsub

‎prow/cmd/tide/OWNERS

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- cjwagner
65
approvers:
7-
- chaodaiG
86
- cjwagner
7+
emeritus_approvers:
8+
- chaodaiG
99
labels:
1010
- area/prow/tide

‎prow/crier/OWNERS

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,10 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22
reviewers:
3-
- chaodaiG
43
- cjwagner
54
approvers:
6-
- chaodaiG
75
- cjwagner
86
emeritus_approvers:
7+
- chaodaiG
98
- fejta
109
- krzyzacy
1110
- Katharine

‎prow/deck/OWNERS

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22
reviewers:
3-
- chaodaiG
43
- michelle192837
54
- mpherman2
65
- smg247
76
approvers:
8-
- chaodaiG
97
- michelle192837
108
- mpherman2
119
- smg247
10+
emeritus_approvers:
11+
- chaodaiG
1212
labels:
1313
- area/prow/deck

‎prow/gerrit/OWNERS

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- chases2
65
approvers:
7-
- chaodaiG
86
- chases2
97
emeritus_approvers:
8+
- chaodaiG
109
- fejta
1110
- Katharine
1211
labels:

‎prow/plank/OWNERS

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- cjwagner
65
approvers:
7-
- chaodaiG
86
- cjwagner
7+
emeritus_approvers:
8+
- chaodaiG
99
labels:
1010
- area/prow/plank

‎prow/pubsub/OWNERS

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,13 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
approvers:
4-
- chaodaiG
54
- cjwagner
65
- listx
76
reviewers:
8-
- chaodaiG
97
- cjwagner
108
- listx
119
emeritus_approvers:
10+
- chaodaiG
1211
- sebastienvas
1312
- yutongz
1413
labels:

‎prow/spyglass/OWNERS

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,15 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- michelle192837
65
- mpherman2
76
- smg247
87
approvers:
9-
- chaodaiG
108
- michelle192837
119
- mpherman2
1210
- smg247
1311
emeritus_approvers:
12+
- chaodaiG
1413
- Katharine
1514
- paulangton
1615
labels:

‎prow/test/integration/OWNERS

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- listx
65
approvers:
7-
- chaodaiG
86
- listx
7+
emeritus_approvers:
8+
- chaodaiG

‎prow/tide/OWNERS

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# See the OWNERS docs at https://go.k8s.io/owners
22

33
reviewers:
4-
- chaodaiG
54
- cjwagner
65
approvers:
7-
- chaodaiG
86
- cjwagner
7+
emeritus_approvers:
8+
- chaodaiG
99
labels:
1010
- area/prow/tide

0 commit comments

Comments
 (0)