diff --git a/ci-operator/config/openshift/api/OWNERS b/ci-operator/config/openshift/api/OWNERS index 6496c8f9dafde..328fac17c2da5 100644 --- a/ci-operator/config/openshift/api/OWNERS +++ b/ci-operator/config/openshift/api/OWNERS @@ -1,5 +1,7 @@ -reviewers: +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/api/blob/5ad8479f64f1b60ee9c62ce8ef1fe6638838725e/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - smarterclayton - - deads2k - - liggitt +- smarterclayton +- deads2k diff --git a/ci-operator/config/openshift/azure-misc/OWNERS b/ci-operator/config/openshift/azure-misc/OWNERS index 0dfee8912216a..29732778caf09 100644 --- a/ci-operator/config/openshift/azure-misc/OWNERS +++ b/ci-operator/config/openshift/azure-misc/OWNERS @@ -1,13 +1,9 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/azure-misc/blob/7d326ed11476949c1f70ff7ecaf161a3808f046e/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: -- jim-minter -- kargakis -- kwoodson -- mjudeikis - pweil- -reviewers: - jim-minter - kargakis - kwoodson -- mjudeikis -- pweil- -- thanasisk diff --git a/ci-operator/config/openshift/ci-operator-prowgen/OWNERS b/ci-operator/config/openshift/ci-operator-prowgen/OWNERS index 28370719d26ca..756e5e2bb0e4b 100644 --- a/ci-operator/config/openshift/ci-operator-prowgen/OWNERS +++ b/ci-operator/config/openshift/ci-operator-prowgen/OWNERS @@ -1,2 +1,6 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/ci-operator-prowgen/blob/ca7e5db8cd812a247b1bf254f79c3f9efa5637bb/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - petr-muller diff --git a/ci-operator/config/openshift/ci-operator/OWNERS b/ci-operator/config/openshift/ci-operator/OWNERS index 3d86e4ef1e9c5..ff8a367ab4a68 100644 --- a/ci-operator/config/openshift/ci-operator/OWNERS +++ b/ci-operator/config/openshift/ci-operator/OWNERS @@ -1,3 +1,7 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/ci-operator/blob/7dcbc7f59a4b2630dbc23a436152da5f1f0b0aca/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - smarterclayton - stevekuznetsov diff --git a/ci-operator/config/openshift/cluster-api-provider-aws/OWNERS b/ci-operator/config/openshift/cluster-api-provider-aws/OWNERS index b098aa7764857..19193bcfd3795 100644 --- a/ci-operator/config/openshift/cluster-api-provider-aws/OWNERS +++ b/ci-operator/config/openshift/cluster-api-provider-aws/OWNERS @@ -1,9 +1,11 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/cluster-api-provider-aws/blob/06c0d2cc2e6f700a1c0e38332cbbc64ed9360832/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: - - enxebre - - frobware - - ingvagabund - - paulfantom - - spangenberg - - trawler +- enxebre +- frobware +- ingvagabund +- paulfantom +- spangenberg +- trawler diff --git a/ci-operator/config/openshift/cluster-dns-operator/OWNERS b/ci-operator/config/openshift/cluster-dns-operator/OWNERS index aaa37f85eccc7..ac92e5bbcf256 100644 --- a/ci-operator/config/openshift/cluster-dns-operator/OWNERS +++ b/ci-operator/config/openshift/cluster-dns-operator/OWNERS @@ -1,10 +1,14 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/cluster-dns-operator/blob/b8a2d7083b202501ca60aca1c81ce50f7adbbce6/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - ironcladlou - - knobunc +- ironcladlou +- knobunc reviewers: - - ironcladlou - - knobunc - - pravisankar - - imcsk8 - - ramr - - Miciah +- ironcladlou +- knobunc +- pravisankar +- imcsk8 +- ramr +- Miciah diff --git a/ci-operator/config/openshift/cluster-ingress-operator/OWNERS b/ci-operator/config/openshift/cluster-ingress-operator/OWNERS index aaa37f85eccc7..156ab822c8049 100644 --- a/ci-operator/config/openshift/cluster-ingress-operator/OWNERS +++ b/ci-operator/config/openshift/cluster-ingress-operator/OWNERS @@ -1,10 +1,14 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/cluster-ingress-operator/blob/71096460ca4986f646e9593493b845b026b93f3c/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - ironcladlou - - knobunc +- ironcladlou +- knobunc reviewers: - - ironcladlou - - knobunc - - pravisankar - - imcsk8 - - ramr - - Miciah +- ironcladlou +- knobunc +- pravisankar +- imcsk8 +- ramr +- Miciah diff --git a/ci-operator/config/openshift/cluster-monitoring-operator/OWNERS b/ci-operator/config/openshift/cluster-monitoring-operator/OWNERS index d566dff12ffbb..10e8cfdf76e14 100644 --- a/ci-operator/config/openshift/cluster-monitoring-operator/OWNERS +++ b/ci-operator/config/openshift/cluster-monitoring-operator/OWNERS @@ -1,12 +1,15 @@ -reviewers: +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/cluster-monitoring-operator/blob/2500badeddb476785f04b4a0fe6dcd375c8686fb/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + +approvers: - brancz - mxinden - elad661 - ironcladlou - squat - s-urbaniak - -approvers: +reviewers: - brancz - mxinden - elad661 diff --git a/ci-operator/config/openshift/cluster-operator/OWNERS b/ci-operator/config/openshift/cluster-operator/OWNERS deleted file mode 100644 index 4b766834ca296..0000000000000 --- a/ci-operator/config/openshift/cluster-operator/OWNERS +++ /dev/null @@ -1,7 +0,0 @@ -approvers: -- csrwng -- dgoodwin -- stevekuznetsov -- joelddiaz -- twiest -- abutcher diff --git a/ci-operator/config/openshift/cluster-samples-operator/OWNERS b/ci-operator/config/openshift/cluster-samples-operator/OWNERS index 18e8e6ab9c23a..c42d35f0989d2 100644 --- a/ci-operator/config/openshift/cluster-samples-operator/OWNERS +++ b/ci-operator/config/openshift/cluster-samples-operator/OWNERS @@ -1,3 +1,7 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/cluster-samples-operator/blob/4290ddd3fb695141eb33e0ae2aa795fea0e709ab/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - bparees - gabemontero diff --git a/ci-operator/config/openshift/cluster-version-operator/OWNERS b/ci-operator/config/openshift/cluster-version-operator/OWNERS index edeceee355118..80e5e10a1c9ea 100644 --- a/ci-operator/config/openshift/cluster-version-operator/OWNERS +++ b/ci-operator/config/openshift/cluster-version-operator/OWNERS @@ -1,9 +1,11 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/cluster-version-operator/blob/7fd23584325138b3a6420cc2a9a0e253233a8637/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: - - crawford - - smarterclayton - - yifan-gu +- crawford +- smarterclayton +- yifan-gu reviewers: - - abhinavdahiya - - wking +- abhinavdahiya +- wking diff --git a/ci-operator/config/openshift/image-registry/OWNERS b/ci-operator/config/openshift/image-registry/OWNERS index e254ff7fba15c..4f5935641da33 100644 --- a/ci-operator/config/openshift/image-registry/OWNERS +++ b/ci-operator/config/openshift/image-registry/OWNERS @@ -1,16 +1,20 @@ -reviewers: - - smarterclayton - - deads2k - - liggitt - - bparees - - dmage - - legionus - - miminar +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/image-registry/blob/0d49798e519cb36d27c97392e92a9bf41ef90b66/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - smarterclayton - - deads2k - - liggitt - - bparees - - dmage - - legionus - - miminar +- smarterclayton +- deads2k +- liggitt +- bparees +- dmage +- legionus +- miminar +reviewers: +- smarterclayton +- deads2k +- liggitt +- bparees +- dmage +- legionus +- miminar diff --git a/ci-operator/config/openshift/installer/OWNERS b/ci-operator/config/openshift/installer/OWNERS index ead2bf30b1fcc..306d1cbfa59b0 100644 --- a/ci-operator/config/openshift/installer/OWNERS +++ b/ci-operator/config/openshift/installer/OWNERS @@ -1,11 +1,13 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/installer/blob/1823ec269d235ff5e99688f57486f937999cab79/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: - - aaronlevy - - abhinavdahiya - - crawford - - smarterclayton - - wking - - yifan-gu +- aaronlevy +- abhinavdahiya +- crawford +- smarterclayton +- wking +- yifan-gu reviewers: - - vikramsk +- vikramsk diff --git a/ci-operator/config/openshift/jenkins/OWNERS b/ci-operator/config/openshift/jenkins/OWNERS index b134666c03234..023a5eba13ec5 100644 --- a/ci-operator/config/openshift/jenkins/OWNERS +++ b/ci-operator/config/openshift/jenkins/OWNERS @@ -1,3 +1,7 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/jenkins/blob/5c3ee917ede2796be1a324a29ceb54d128007daa/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - bparees -- gabemontero \ No newline at end of file +- gabemontero diff --git a/ci-operator/config/openshift/kube-state-metrics/OWNERS b/ci-operator/config/openshift/kube-state-metrics/OWNERS index 613fd600823f1..7cc634d0af20d 100644 --- a/ci-operator/config/openshift/kube-state-metrics/OWNERS +++ b/ci-operator/config/openshift/kube-state-metrics/OWNERS @@ -1,7 +1,11 @@ -reviewers: - - brancz - - andyxning +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/kube-state-metrics/blob/3e5e2d70f1527db07c20f7dd747a57e971c592a9/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - brancz - - andyxning - - fabxc +- brancz +- andyxning +- fabxc +reviewers: +- brancz +- andyxning diff --git a/ci-operator/config/openshift/machine-api-operator/OWNERS b/ci-operator/config/openshift/machine-api-operator/OWNERS index 898030684fdeb..ec849418f8f5b 100644 --- a/ci-operator/config/openshift/machine-api-operator/OWNERS +++ b/ci-operator/config/openshift/machine-api-operator/OWNERS @@ -1,7 +1,12 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/machine-api-operator/blob/429541465bef1e0b1b4be025f342c665c0479c0a/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - enxebre - - frobware - - ingvagabund - - paulfantom - - spangenberg - - trawler +- enxebre +- frobware +- ingvagabund +- paulfantom +- spangenberg +- trawler +- bison diff --git a/ci-operator/config/openshift/machine-config-operator/OWNERS b/ci-operator/config/openshift/machine-config-operator/OWNERS index ef094ac44e02d..f79efbafc0c3a 100644 --- a/ci-operator/config/openshift/machine-config-operator/OWNERS +++ b/ci-operator/config/openshift/machine-config-operator/OWNERS @@ -1,8 +1,10 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/machine-config-operator/blob/1a4fb283860a1434ee5c9721a21ebdf20374bb90/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: - - abhinavdahiya - - crawford - - smarterclayton - - wking - - yifan-gu +- abhinavdahiya +- crawford +- smarterclayton +- wking +- yifan-gu diff --git a/ci-operator/config/openshift/openshift-ansible/OWNERS b/ci-operator/config/openshift/openshift-ansible/OWNERS index e7adf39db00bc..5901beb5cc3eb 100644 --- a/ci-operator/config/openshift/openshift-ansible/OWNERS +++ b/ci-operator/config/openshift/openshift-ansible/OWNERS @@ -1,12 +1,14 @@ -# approval == this is a good idea /approve +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/openshift-ansible/blob/33d278ada1a6351fa787210c80981b8791b9eda1/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - michaelgugino - - mtnbikenc - - sdodson - - vrutkovs -# review == this code is good /lgtm +- michaelgugino +- mtnbikenc +- sdodson +- vrutkovs reviewers: - - michaelgugino - - mtnbikenc - - sdodson - - vrutkovs +- michaelgugino +- mtnbikenc +- sdodson +- vrutkovs diff --git a/ci-operator/config/openshift/openshift-azure/OWNERS b/ci-operator/config/openshift/openshift-azure/OWNERS index 0dfee8912216a..d73da071b6c73 100644 --- a/ci-operator/config/openshift/openshift-azure/OWNERS +++ b/ci-operator/config/openshift/openshift-azure/OWNERS @@ -1,3 +1,7 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/openshift-azure/blob/33ff2171571945b9f87cc9e8132f7da826bce3e8/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - jim-minter - kargakis @@ -5,9 +9,12 @@ approvers: - mjudeikis - pweil- reviewers: +- amanohar - jim-minter - kargakis - kwoodson - mjudeikis - pweil- +- shrutir25 +- sozercan - thanasisk diff --git a/ci-operator/config/openshift/origin-aggregated-logging/OWNERS b/ci-operator/config/openshift/origin-aggregated-logging/OWNERS index f860e14133eee..63f57a9d946ce 100644 --- a/ci-operator/config/openshift/origin-aggregated-logging/OWNERS +++ b/ci-operator/config/openshift/origin-aggregated-logging/OWNERS @@ -1,16 +1,19 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/origin-aggregated-logging/blob/810fc241c2d65ecf3396bd3721be1540ca606586/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - jwforres - - jcantrill - - ewolinetz - - richm - - lukas-vlcek - - nhosoi - - josefkarasek - - nkinder +- jwforres +- jcantrill +- ewolinetz +- richm +- lukas-vlcek +- nhosoi +- josefkarasek +- nkinder reviewers: - - jwforres - - jcantrill - - ewolinetz - - richm - - nhosoi +- jwforres +- jcantrill +- ewolinetz +- richm +- nhosoi diff --git a/ci-operator/config/openshift/origin/OWNERS b/ci-operator/config/openshift/origin/OWNERS index e04702d5dc15a..fdcb369251a01 100644 --- a/ci-operator/config/openshift/origin/OWNERS +++ b/ci-operator/config/openshift/origin/OWNERS @@ -1,19 +1,21 @@ -reviewers: - - smarterclayton - - deads2k - - liggitt - - bparees - - stevekuznetsov - - soltysh - - tbielawa +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/origin/blob/546d968652201fab0107b0fc3cd651ae766a8c8b/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - smarterclayton - - deads2k - - liggitt - - bparees - - mfojtik - - eparis - - derekwaynecarr - - stevekuznetsov # for build and tooling changes - - tbielawa # also for build and automated release tooling changes - - pweil- +- smarterclayton +- deads2k +- bparees +- mfojtik +- eparis +- derekwaynecarr +- stevekuznetsov +- tbielawa +- pweil- +reviewers: +- smarterclayton +- deads2k +- bparees +- stevekuznetsov +- soltysh +- tbielawa diff --git a/ci-operator/config/openshift/service-catalog/OWNERS b/ci-operator/config/openshift/service-catalog/OWNERS index 35724d14e2c76..3df0df72d7462 100644 --- a/ci-operator/config/openshift/service-catalog/OWNERS +++ b/ci-operator/config/openshift/service-catalog/OWNERS @@ -1,4 +1,9 @@ +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/service-catalog/blob/df5a8b34b6f66cf539dcf45e0827814df959348f/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - jpeeler - pmorie - jboyd01 +- luksa diff --git a/ci-operator/config/openshift/service-serving-cert-signer/OWNERS b/ci-operator/config/openshift/service-serving-cert-signer/OWNERS index 27fbbf15ce7be..85716c8cfad2a 100644 --- a/ci-operator/config/openshift/service-serving-cert-signer/OWNERS +++ b/ci-operator/config/openshift/service-serving-cert-signer/OWNERS @@ -1,11 +1,15 @@ -reviewers: - - deads2k - - enj - - mfojtik - - mrogers950 - - simo5 - - soltysh +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/service-serving-cert-signer/blob/c2e19f18e28faa64433bc7ef634325eed8cd8a45/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + approvers: - - smarterclayton - - deads2k - - liggitt +- smarterclayton +- deads2k +- liggitt +reviewers: +- deads2k +- enj +- mfojtik +- mrogers950 +- simo5 +- soltysh diff --git a/ci-operator/config/openshift/telemeter/OWNERS b/ci-operator/config/openshift/telemeter/OWNERS index 0a2daaa98ad2a..265ce6aca1224 100644 --- a/ci-operator/config/openshift/telemeter/OWNERS +++ b/ci-operator/config/openshift/telemeter/OWNERS @@ -1,11 +1,14 @@ -reviewers: +# DO NOT EDIT; this file is auto-generated using tools/populate-owners. +# from https://github.com/openshift/telemeter/blob/465bdd9c8b8e4449a63b20dea82cbd4728b2fb2c/OWNERS +# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md + +approvers: - brancz - mxinden - squat - s-urbaniak - smarterclayton - -approvers: +reviewers: - brancz - mxinden - squat diff --git a/ci-operator/jobs/kubernetes-incubator/cri-o/OWNERS b/ci-operator/jobs/kubernetes-incubator/cri-o/OWNERS index e6c03d9bdd7f9..9fe872e15ff58 100644 --- a/ci-operator/jobs/kubernetes-incubator/cri-o/OWNERS +++ b/ci-operator/jobs/kubernetes-incubator/cri-o/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/kubernetes-incubator/cri-o/blob/3fc75c50160907073e161d78fc51a5ed5dad8313/OWNERS +# from https://github.com/kubernetes-incubator/cri-o/blob/9246d35b40666132a27b89bfd2c5b9e3eef55a8b/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/api/OWNERS b/ci-operator/jobs/openshift/api/OWNERS index dbf904b7a2d32..328fac17c2da5 100644 --- a/ci-operator/jobs/openshift/api/OWNERS +++ b/ci-operator/jobs/openshift/api/OWNERS @@ -1,8 +1,7 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/api/blob/bccfa651752a8a9f963b9899b477dcffbe9be786/OWNERS +# from https://github.com/openshift/api/blob/5ad8479f64f1b60ee9c62ce8ef1fe6638838725e/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: - smarterclayton - deads2k -- liggitt diff --git a/ci-operator/jobs/openshift/ci-operator/OWNERS b/ci-operator/jobs/openshift/ci-operator/OWNERS index 3671cd7c8494c..ff8a367ab4a68 100644 --- a/ci-operator/jobs/openshift/ci-operator/OWNERS +++ b/ci-operator/jobs/openshift/ci-operator/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/ci-operator/blob/4504de72562a88e1b0cf3c018a5ee78164695017/OWNERS +# from https://github.com/openshift/ci-operator/blob/7dcbc7f59a4b2630dbc23a436152da5f1f0b0aca/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/cluster-api-provider-aws/OWNERS b/ci-operator/jobs/openshift/cluster-api-provider-aws/OWNERS index 12ff7e46569c5..19193bcfd3795 100644 --- a/ci-operator/jobs/openshift/cluster-api-provider-aws/OWNERS +++ b/ci-operator/jobs/openshift/cluster-api-provider-aws/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/cluster-api-provider-aws/blob/44e974de04f496f9552ecd37b73cad01b6d69f4d/OWNERS +# from https://github.com/openshift/cluster-api-provider-aws/blob/06c0d2cc2e6f700a1c0e38332cbbc64ed9360832/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/cluster-ingress-operator/OWNERS b/ci-operator/jobs/openshift/cluster-ingress-operator/OWNERS index 901081304fa7c..156ab822c8049 100644 --- a/ci-operator/jobs/openshift/cluster-ingress-operator/OWNERS +++ b/ci-operator/jobs/openshift/cluster-ingress-operator/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/cluster-ingress-operator/blob/387220e8f17b04ee237d3883a0913790b59e0194/OWNERS +# from https://github.com/openshift/cluster-ingress-operator/blob/71096460ca4986f646e9593493b845b026b93f3c/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/cluster-samples-operator/OWNERS b/ci-operator/jobs/openshift/cluster-samples-operator/OWNERS index e8ed28726b90d..c42d35f0989d2 100644 --- a/ci-operator/jobs/openshift/cluster-samples-operator/OWNERS +++ b/ci-operator/jobs/openshift/cluster-samples-operator/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/cluster-samples-operator/blob/eccdb3ca00baec2b280c2ebb12a7c56edec7f818/OWNERS +# from https://github.com/openshift/cluster-samples-operator/blob/4290ddd3fb695141eb33e0ae2aa795fea0e709ab/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/installer/OWNERS b/ci-operator/jobs/openshift/installer/OWNERS index dae8bc29d096c..306d1cbfa59b0 100644 --- a/ci-operator/jobs/openshift/installer/OWNERS +++ b/ci-operator/jobs/openshift/installer/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/installer/blob/ae7fe33afbf0bf3b19bf77b0ce26ffb4c98cfe11/OWNERS +# from https://github.com/openshift/installer/blob/1823ec269d235ff5e99688f57486f937999cab79/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/jenkins-client-plugin/OWNERS b/ci-operator/jobs/openshift/jenkins-client-plugin/OWNERS index 53c5c5fd3c632..37065ae233d8b 100644 --- a/ci-operator/jobs/openshift/jenkins-client-plugin/OWNERS +++ b/ci-operator/jobs/openshift/jenkins-client-plugin/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/jenkins-client-plugin/blob/ac38bc1582f874f316eea623e0caad1dba482806/OWNERS +# from https://github.com/openshift/jenkins-client-plugin/blob/abfb53fe415b530be2221945c108ff275d00228b/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/jenkins-plugin/OWNERS b/ci-operator/jobs/openshift/jenkins-plugin/OWNERS index 9065d54ed5d0d..7fcdf4b51396b 100644 --- a/ci-operator/jobs/openshift/jenkins-plugin/OWNERS +++ b/ci-operator/jobs/openshift/jenkins-plugin/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/jenkins-plugin/blob/ed4cf1ac84f8456e6844f5c14a08f7bcb94afe16/OWNERS +# from https://github.com/openshift/jenkins-plugin/blob/5998b456a949718153b9663a43bd18235fb40ed6/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/machine-api-operator/OWNERS b/ci-operator/jobs/openshift/machine-api-operator/OWNERS index ec452aba4604a..ec849418f8f5b 100644 --- a/ci-operator/jobs/openshift/machine-api-operator/OWNERS +++ b/ci-operator/jobs/openshift/machine-api-operator/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/machine-api-operator/blob/2f4a03bfff35c8d0bf3fbf5b7804e89a637699f1/OWNERS +# from https://github.com/openshift/machine-api-operator/blob/429541465bef1e0b1b4be025f342c665c0479c0a/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: @@ -9,3 +9,4 @@ approvers: - paulfantom - spangenberg - trawler +- bison diff --git a/ci-operator/jobs/openshift/machine-config-operator/OWNERS b/ci-operator/jobs/openshift/machine-config-operator/OWNERS index ad91b240365ba..f79efbafc0c3a 100644 --- a/ci-operator/jobs/openshift/machine-config-operator/OWNERS +++ b/ci-operator/jobs/openshift/machine-config-operator/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/machine-config-operator/blob/a61de6ce1182f893ea69d0c6a38930dd1b3278df/OWNERS +# from https://github.com/openshift/machine-config-operator/blob/1a4fb283860a1434ee5c9721a21ebdf20374bb90/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/online-registration/OWNERS b/ci-operator/jobs/openshift/online-registration/OWNERS index 9f8d038a666e1..396dea7a197ae 100644 --- a/ci-operator/jobs/openshift/online-registration/OWNERS +++ b/ci-operator/jobs/openshift/online-registration/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/online-registration/blob/1149b551e1708054d0c5c3a58641127cb30506fa/OWNERS +# from https://github.com/openshift/online-registration/blob/4fe825943a2c9a1270ad69b6e55d0cefa2ca7e44/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/openshift-ansible/OWNERS b/ci-operator/jobs/openshift/openshift-ansible/OWNERS index 5c60122cc5df1..5901beb5cc3eb 100644 --- a/ci-operator/jobs/openshift/openshift-ansible/OWNERS +++ b/ci-operator/jobs/openshift/openshift-ansible/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/openshift-ansible/blob/35efde925809ef23e9a006681829417f1e3e515f/OWNERS +# from https://github.com/openshift/openshift-ansible/blob/33d278ada1a6351fa787210c80981b8791b9eda1/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/openshift-azure/OWNERS b/ci-operator/jobs/openshift/openshift-azure/OWNERS index 129deed7c7aac..d73da071b6c73 100644 --- a/ci-operator/jobs/openshift/openshift-azure/OWNERS +++ b/ci-operator/jobs/openshift/openshift-azure/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/openshift-azure/blob/6f486a0ad4184b4f5703764572c627cec17d8b75/OWNERS +# from https://github.com/openshift/openshift-azure/blob/33ff2171571945b9f87cc9e8132f7da826bce3e8/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/origin-aggregated-logging/OWNERS b/ci-operator/jobs/openshift/origin-aggregated-logging/OWNERS index 75a2c6327a624..63f57a9d946ce 100644 --- a/ci-operator/jobs/openshift/origin-aggregated-logging/OWNERS +++ b/ci-operator/jobs/openshift/origin-aggregated-logging/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/origin-aggregated-logging/blob/81901b7ab61f49d4660863594886c33c5eba54d7/OWNERS +# from https://github.com/openshift/origin-aggregated-logging/blob/810fc241c2d65ecf3396bd3721be1540ca606586/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/ci-operator/jobs/openshift/origin/OWNERS b/ci-operator/jobs/openshift/origin/OWNERS index e95f3db9631e4..fdcb369251a01 100644 --- a/ci-operator/jobs/openshift/origin/OWNERS +++ b/ci-operator/jobs/openshift/origin/OWNERS @@ -1,11 +1,10 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/origin/blob/b0069e668a5d0413d3360c8eaf1fe425f83cd104/OWNERS +# from https://github.com/openshift/origin/blob/546d968652201fab0107b0fc3cd651ae766a8c8b/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: - smarterclayton - deads2k -- liggitt - bparees - mfojtik - eparis @@ -16,7 +15,6 @@ approvers: reviewers: - smarterclayton - deads2k -- liggitt - bparees - stevekuznetsov - soltysh diff --git a/ci-operator/jobs/openshift/release/OWNERS b/ci-operator/jobs/openshift/release/OWNERS index 6e0837a1e84dd..6f2f1d047829f 100644 --- a/ci-operator/jobs/openshift/release/OWNERS +++ b/ci-operator/jobs/openshift/release/OWNERS @@ -1,5 +1,5 @@ # DO NOT EDIT; this file is auto-generated using tools/populate-owners. -# from https://github.com/openshift/release/blob/c4e5ab33800e232c770187b132f096ebafbfccfe/OWNERS +# from https://github.com/openshift/release/blob/8f42984b2e9c47a8113ab714df7e6cc7a942e76c/OWNERS # See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md approvers: diff --git a/tools/populate-owners/README.md b/tools/populate-owners/README.md index d3128c2ab161f..7bbbc94801ce3 100644 --- a/tools/populate-owners/README.md +++ b/tools/populate-owners/README.md @@ -23,8 +23,8 @@ For example, if [openshift/origin][] and [openshift/installer][] both defined an After namespacing aliases, the utility writes `OWNERS_ALIASES` to the root of this repository. If no upstreams define aliases, then the utility removes `OWNER_ALIASES` from the root of this repository. -The utility also iterates through the `ci-operator/jobs/{organization}/{repository}` directories, writing `OWNERS` to reflect the upstream configuration. -If the upstream did not have an `OWNERS` file, the utility removes the associated `ci-operator/jobs/{organization}/{repository}/OWNERS`. +The utility also iterates through the `ci-operator/jobs/{organization}/{repository}` and `ci-operator/config/{organization}/{repository}` directories, writing `OWNERS` to reflect the upstream configuration. +If the upstream did not have an `OWNERS` file, the utility removes the associated `ci-operator/*/{organization}/{repository}/OWNERS`. [openshift/origin]: https://github.com/openshift/origin [openshift/installer]: https://github.com/openshift/installer diff --git a/tools/populate-owners/main.go b/tools/populate-owners/main.go index b3adc39f49210..fcde596f0fc79 100644 --- a/tools/populate-owners/main.go +++ b/tools/populate-owners/main.go @@ -93,6 +93,20 @@ func orgRepos(dir string) (orgRepos []*orgRepo, err error) { return orgRepos, err } +func (orgRepo *orgRepo) getConfig(dir string) (err error) { + path := filepath.Join(dir, orgRepo.Organization, orgRepo.Repository) + info, err := os.Stat(path) + if err != nil { + return err + } + + if info.IsDir() { + orgRepo.Directories = append(orgRepo.Directories, path) + } + + return nil +} + func (orgRepo *orgRepo) getOwners() (err error) { dir, err := ioutil.TempDir("", "populate-owners-") if err != nil { @@ -296,8 +310,14 @@ func pullOwners(directory string) (err error) { return err } + config := filepath.Join(repoRoot, "ci-operator", "config") for _, orgRepo := range orgRepos { - err := orgRepo.getOwners() + err = orgRepo.getConfig(config) + if err != nil && !os.IsNotExist(err) { + return err + } + + err = orgRepo.getOwners() if err != nil && !os.IsNotExist(err) { return err } diff --git a/tools/populate-owners/main_test.go b/tools/populate-owners/main_test.go index a084aef8e0e5b..333a46a432f25 100644 --- a/tools/populate-owners/main_test.go +++ b/tools/populate-owners/main_test.go @@ -92,6 +92,68 @@ func TestOrgRepos(t *testing.T) { assertEqual(t, orgRepos, expected) } +func TestGetConfig(t *testing.T) { + dir, err := ioutil.TempDir("", "populate-owners-") + if err != nil { + t.Fatal(err) + } + defer os.RemoveAll(dir) + + repoAB := filepath.Join(dir, "a", "b") + err = os.MkdirAll(repoAB, 0777) + if err != nil { + t.Fatal(err) + } + + for _, test := range []struct{ + name string + input *orgRepo + expected *orgRepo + error *regexp.Regexp + }{ + { + name: "config exists", + input: &orgRepo{ + Directories: []string{"some/directory"}, + Organization: "a", + Repository: "b", + }, + expected: &orgRepo{ + Directories: []string{"some/directory", filepath.Join(dir, "a", "b")}, + Organization: "a", + Repository: "b", + }, + }, + { + name: "config does not exist", + input: &orgRepo{ + Directories: []string{"some/directory"}, + Organization: "c", + Repository: "d", + }, + expected: &orgRepo{ + Directories: []string{"some/directory"}, + Organization: "c", + Repository: "d", + }, + error: regexp.MustCompile("^stat .*/c/d: no such file or directory"), + }, + } { + t.Run(test.name, func (t *testing.T) { + err := test.input.getConfig(dir) + if test.error == nil { + if err != nil { + t.Fatal(err) + } + } else if !test.error.MatchString(err.Error()) { + t.Fatalf("unexpected error: %v does not match %v", err, test.error) + } + + assertEqual(t, test.input, test.expected) + }) + } +} + func TestExtractOwners(t *testing.T) { dir, err := ioutil.TempDir("", "populate-owners-") if err != nil {