Browse Source

Add OWNERS files

Attempt to give component owners autonomy and ensure that PRs
automatically receive assigned reviewers.

Just a first pass, open to changes in the future
Scott Dodson 7 years ago
parent
commit
1a656ce0b2
47 changed files with 570 additions and 0 deletions
  1. 12 0
      OWNERS
  2. 3 0
      docs/openshift_components.md
  3. 16 0
      playbooks/aws/OWNERS
  4. 10 0
      playbooks/cluster-operator/OWNERS
  5. 14 0
      playbooks/gcp/OWNERS
  6. 14 0
      playbooks/openshift-checks/OWNERS
  7. 10 0
      playbooks/openshift-descheduler/OWNERS
  8. 14 0
      playbooks/openshift-glusterfs/OWNERS
  9. 12 0
      playbooks/openshift-logging/OWNERS
  10. 16 0
      playbooks/openshift-metrics/OWNERS
  11. 12 0
      playbooks/openshift-monitoring/OWNERS
  12. 10 0
      playbooks/openshift-node-problem-detector/OWNERS
  13. 16 0
      playbooks/openshift-prometheus/OWNERS
  14. 16 0
      playbooks/openshift-service-catalog/OWNERS
  15. 14 0
      playbooks/openshift-web-console/OWNERS
  16. 8 0
      playbooks/openstack/OWNERS
  17. 18 0
      roles/ansible_service_broker/OWNERS
  18. 8 0
      roles/kuryr/OWNERS
  19. 16 0
      roles/openshift_aws/OWNERS
  20. 14 0
      roles/openshift_builddefaults/OWNERS
  21. 14 0
      roles/openshift_buildoverrides/OWNERS
  22. 14 0
      roles/openshift_ca/OWNERS
  23. 14 0
      roles/openshift_certificate_expiry/OWNERS
  24. 12 0
      roles/openshift_cfme/OWNERS
  25. 12 0
      roles/openshift_cluster_monitoring_operator/OWNERS
  26. 16 0
      roles/openshift_daemonset_config/OWNERS
  27. 10 0
      roles/openshift_descheduler/OWNERS
  28. 10 0
      roles/openshift_docker_gc/OWNERS
  29. 14 0
      roles/openshift_gcp/OWNERS
  30. 14 0
      roles/openshift_health_checker/OWNERS
  31. 12 0
      roles/openshift_logging/OWNERS
  32. 11 0
      roles/openshift_logging_curator/OWNERS
  33. 11 0
      roles/openshift_logging_elasticsearch/OWNERS
  34. 11 0
      roles/openshift_logging_eventrouter/OWNERS
  35. 11 0
      roles/openshift_logging_fluentd/OWNERS
  36. 11 0
      roles/openshift_logging_kibana/OWNERS
  37. 12 0
      roles/openshift_logging_mux/OWNERS
  38. 10 0
      roles/openshift_metrics/OWNERS
  39. 14 0
      roles/openshift_named_certificates/OWNERS
  40. 14 0
      roles/openshift_node_certificates/OWNERS
  41. 10 0
      roles/openshift_node_problem_detector/OWNERS
  42. 8 0
      roles/openshift_openstack/OWNERS
  43. 8 0
      roles/openshift_prometheus/OWNERS
  44. 16 0
      roles/openshift_service_catalog/OWNERS
  45. 10 0
      roles/openshift_storage_glusterfs/OWNERS
  46. 10 0
      roles/openshift_web_console/OWNERS
  47. 8 0
      roles/tuned/OWNERS

+ 12 - 0
OWNERS

@@ -0,0 +1,12 @@
+# approval == this is a good idea /approve
+approvers:
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 3 - 0
docs/openshift_components.md

@@ -7,6 +7,7 @@
 Components in OpenShift-Ansible consist of two main parts:
 Components in OpenShift-Ansible consist of two main parts:
 * Entry point playbook(s)
 * Entry point playbook(s)
 * Ansible role
 * Ansible role
+* OWNERS files in both the playbooks and roles associated with the component
 
 
 When writing playbooks and roles, follow these basic guidelines to ensure
 When writing playbooks and roles, follow these basic guidelines to ensure
 success and maintainability. 
 success and maintainability. 
@@ -48,6 +49,7 @@ playbooks/openshift-component_name
 ├── private
 ├── private
 │   ├── config.yml                      Included by the Cluster Installer
 │   ├── config.yml                      Included by the Cluster Installer
 │   └── roles -> ../../roles            Don't forget to create this symlink
 │   └── roles -> ../../roles            Don't forget to create this symlink
+├── OWNERS                              Assign 2-3 approvers and reviewers
 └── README.md                           Tell us what this component does
 └── README.md                           Tell us what this component does
 ```
 ```
 
 
@@ -108,6 +110,7 @@ roles/openshift_component_name
 │   └── main.yml
 │   └── main.yml
 ├── meta
 ├── meta
 │   └── main.yml
 │   └── main.yml
+├── OWNERS                              Assign 2-3 approvers and reviewers
 ├── README.md
 ├── README.md
 ├── tasks
 ├── tasks
 │   └── main.yml                        Default playbook used when calling the role
 │   └── main.yml                        Default playbook used when calling the role

+ 16 - 0
playbooks/aws/OWNERS

@@ -0,0 +1,16 @@
+# approval == this is a good idea /approve
+approvers:
+  - kwoodson
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - kwoodson
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 10 - 0
playbooks/cluster-operator/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - abutcher
+  - dgoodwin
+  - csrwng
+# review == this code is good /lgtm
+reviewers:
+  - abutcher
+  - dgoodwin
+  - csrwng

+ 14 - 0
playbooks/gcp/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - smarterclayton
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - smarterclayton
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 14 - 0
playbooks/openshift-checks/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - sosiouxme
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - sosiouxme
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 10 - 0
playbooks/openshift-descheduler/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - aveshagarwal
+  - ingvagabund
+  - ravisantoshgudimetla
+# review == this code is good /lgtm
+reviewers:
+  - aveshagarwal
+  - ingvagabund
+  - ravisantoshgudimetla

+ 14 - 0
playbooks/openshift-glusterfs/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - jarrpa
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - jarrpa
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 12 - 0
playbooks/openshift-logging/OWNERS

@@ -0,0 +1,12 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan

+ 16 - 0
playbooks/openshift-metrics/OWNERS

@@ -0,0 +1,16 @@
+# approval == this is a good idea /approve
+approvers:
+  - ewolinetz
+  - jcantrill
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - ewolinetz
+  - jcantrill
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 12 - 0
playbooks/openshift-monitoring/OWNERS

@@ -0,0 +1,12 @@
+# approval == this is a good idea /approve
+approvers:
+  - ironcladlou
+  - elad661
+  - mxinden
+  - brancz
+# review == this code is good /lgtm
+reviewers:
+  - ironcladlou
+  - elad661
+  - mxinden
+  - brancz

+ 10 - 0
playbooks/openshift-node-problem-detector/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - ironcladlou
+  - joelsmith
+  - ingvagabund
+# review == this code is good /lgtm
+reviewers:
+  - ironcladlou
+  - joelsmith
+  - ingvagabund

+ 16 - 0
playbooks/openshift-prometheus/OWNERS

@@ -0,0 +1,16 @@
+# approval == this is a good idea /approve
+approvers:
+  - zgalor
+  - pgier
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - zgalor
+  - pgier
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 16 - 0
playbooks/openshift-service-catalog/OWNERS

@@ -0,0 +1,16 @@
+# approval == this is a good idea /approve
+approvers:
+  - jpeeler
+  - jboyd01
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - jpeeler
+  - jboyd01
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 14 - 0
playbooks/openshift-web-console/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - spadgett
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - spadgett
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 8 - 0
playbooks/openstack/OWNERS

@@ -0,0 +1,8 @@
+# approval == this is a good idea /approve
+approvers:
+  - tomassedovic
+  - tzumainn
+# review == this code is good /lgtm
+reviewers:
+  - tomassedovic
+  - tzumainn

+ 18 - 0
roles/ansible_service_broker/OWNERS

@@ -0,0 +1,18 @@
+# approval == this is a good idea /approve
+approvers:
+  - fabianvf
+  - dymurray
+  - shawn-hurley
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - fabianvf
+  - dymurray
+  - shawn-hurley
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 8 - 0
roles/kuryr/OWNERS

@@ -0,0 +1,8 @@
+# approval == this is a good idea /approve
+approvers:
+  - tomassedovic
+  - tzumainn
+# review == this code is good /lgtm
+reviewers:
+  - tomassedovic
+  - tzumainn

+ 16 - 0
roles/openshift_aws/OWNERS

@@ -0,0 +1,16 @@
+# approval == this is a good idea /approve
+approvers:
+  - kwoodson
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - kwoodson
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 14 - 0
roles/openshift_builddefaults/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - bparees
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - bparees
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 14 - 0
roles/openshift_buildoverrides/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - bparees
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - bparees
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 14 - 0
roles/openshift_ca/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 14 - 0
roles/openshift_certificate_expiry/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 12 - 0
roles/openshift_cfme/OWNERS

@@ -0,0 +1,12 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - sdodson
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan
+  - zgalor

+ 12 - 0
roles/openshift_cluster_monitoring_operator/OWNERS

@@ -0,0 +1,12 @@
+# approval == this is a good idea /approve
+approvers:
+  - ironcladlou
+  - elad661
+  - mxinden
+  - brancz
+# review == this code is good /lgtm
+reviewers:
+  - ironcladlou
+  - elad661
+  - mxinden
+  - brancz

+ 16 - 0
roles/openshift_daemonset_config/OWNERS

@@ -0,0 +1,16 @@
+# approval == this is a good idea /approve
+approvers:
+  - kwoodson
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - kwoodson
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 10 - 0
roles/openshift_descheduler/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - aveshagarwal
+  - ingvagabund
+  - ravisantoshgudimetla
+# review == this code is good /lgtm
+reviewers:
+  - aveshagarwal
+  - ingvagabund
+  - ravisantoshgudimetla

+ 10 - 0
roles/openshift_docker_gc/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - ashcrow
+  - kwoodson
+  - mrunalp
+# review == this code is good /lgtm
+reviewers:
+  - ashcrow
+  - kwoodson
+  - mrunalp

+ 14 - 0
roles/openshift_gcp/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - smarterclayton
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - smarterclayton
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 14 - 0
roles/openshift_health_checker/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - sosiouxme
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - sosiouxme
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 12 - 0
roles/openshift_logging/OWNERS

@@ -0,0 +1,12 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan

+ 11 - 0
roles/openshift_logging_curator/OWNERS

@@ -0,0 +1,11 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - sdodson
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan

+ 11 - 0
roles/openshift_logging_elasticsearch/OWNERS

@@ -0,0 +1,11 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - sdodson
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan

+ 11 - 0
roles/openshift_logging_eventrouter/OWNERS

@@ -0,0 +1,11 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - sdodson
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan

+ 11 - 0
roles/openshift_logging_fluentd/OWNERS

@@ -0,0 +1,11 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - sdodson
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan

+ 11 - 0
roles/openshift_logging_kibana/OWNERS

@@ -0,0 +1,11 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - sdodson
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan

+ 12 - 0
roles/openshift_logging_mux/OWNERS

@@ -0,0 +1,12 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - richm
+  - wozniakjan

+ 10 - 0
roles/openshift_metrics/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - jcantrill
+  - ewolinetz
+  - jsanda
+# review == this code is good /lgtm
+reviewers:
+  - jcantrill
+  - ewolinetz
+  - jsanda

+ 14 - 0
roles/openshift_named_certificates/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 14 - 0
roles/openshift_node_certificates/OWNERS

@@ -0,0 +1,14 @@
+# approval == this is a good idea /approve
+approvers:
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - abutcher
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 10 - 0
roles/openshift_node_problem_detector/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - ironcladlou
+  - joelsmith
+  - ingvagabund
+# review == this code is good /lgtm
+reviewers:
+  - ironcladlou
+  - joelsmith
+  - ingvagabund

+ 8 - 0
roles/openshift_openstack/OWNERS

@@ -0,0 +1,8 @@
+# approval == this is a good idea /approve
+approvers:
+  - tomassedovic
+  - tzumainn
+# review == this code is good /lgtm
+reviewers:
+  - tomassedovic
+  - tzumainn

+ 8 - 0
roles/openshift_prometheus/OWNERS

@@ -0,0 +1,8 @@
+# approval == this is a good idea /approve
+approvers:
+  - pgier
+  - zgalor
+# review == this code is good /lgtm
+reviewers:
+  - pgier
+  - zgalor

+ 16 - 0
roles/openshift_service_catalog/OWNERS

@@ -0,0 +1,16 @@
+# approval == this is a good idea /approve
+approvers:
+  - jpeeler
+  - jboyd01
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs
+# review == this code is good /lgtm
+reviewers:
+  - jpeeler
+  - jboyd01
+  - michaelgugino
+  - mtnbikenc
+  - sdodson
+  - vrutkovs

+ 10 - 0
roles/openshift_storage_glusterfs/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - jarrpa
+  - abutcher
+  - sdodson
+# review == this code is good /lgtm
+reviewers:
+  - jarrpa
+  - abutcher
+  - sdodson

+ 10 - 0
roles/openshift_web_console/OWNERS

@@ -0,0 +1,10 @@
+# approval == this is a good idea /approve
+approvers:
+  - spadgett
+  - jwforres
+  - sdodson
+# review == this code is good /lgtm
+reviewers:
+  - spadgett
+  - jwforres
+  - sdodson

+ 8 - 0
roles/tuned/OWNERS

@@ -0,0 +1,8 @@
+# approval == this is a good idea /approve
+approvers:
+  - jmencak
+  - jeremyeder
+# review == this code is good /lgtm
+reviewers:
+  - jmencak
+  - jeremyeder