)]}'
{".zuul.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":101,"context_line":"      - tools/gate/playbooks/osh-infra-build.yaml"},{"line_number":102,"context_line":"      - tools/gate/playbooks/osh-infra-deploy-k8s.yaml"},{"line_number":103,"context_line":"    run: tools/gate/playbooks/osh-infra-deploy-charts.yaml"},{"line_number":104,"context_line":"    post-run: tools/gate/playbooks/osh-infra-summarize-deploy.yaml"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"- job:"},{"line_number":107,"context_line":"    name: openstack-helm-infra-ubuntu"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_c69a8bb7","line":104,"range":{"start_line":104,"start_character":35,"end_line":104,"end_character":66},"updated":"2017-12-11 02:20:52.000000000","message":"how about we call it collect-logs? or dump-logs as it was in the original gate scripts. As this is doing a lot more than just summarising the deployments.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"kube-state-metrics/templates/service-controller-manager.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  name: kube-controller-manager-discovery"},{"line_number":25,"context_line":"  labels:"},{"line_number":26,"context_line":"    component: kube-controller-manager"},{"line_number":27,"context_line":"{{ tuple $envAll \"controller-manager\" \"metrics\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" | indent 4 }}"},{"line_number":28,"context_line":"  annotations:"},{"line_number":29,"context_line":"{{ tuple $endpoint $envAll  | include \"helm-toolkit.snippets.prometheus_service_annotations\" | indent 4 }}"},{"line_number":30,"context_line":"spec:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_66efff1d","line":27,"updated":"2017-12-11 02:20:52.000000000","message":"this will declare the `component` key twice (https://github.com/openstack/openstack-helm-infra/blob/master/helm-toolkit/templates/snippets/_kubernetes_metadata_labels.tpl#L23). line 26 should probably be removed? Also is this change in scope of this PS?","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  name: kube-controller-manager-discovery"},{"line_number":25,"context_line":"  labels:"},{"line_number":26,"context_line":"    component: kube-controller-manager"},{"line_number":27,"context_line":"{{ tuple $envAll \"controller-manager\" \"metrics\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" | indent 4 }}"},{"line_number":28,"context_line":"  annotations:"},{"line_number":29,"context_line":"{{ tuple $endpoint $envAll  | include \"helm-toolkit.snippets.prometheus_service_annotations\" | indent 4 }}"},{"line_number":30,"context_line":"spec:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_26d337d7","line":27,"in_reply_to":"df87a7cf_66efff1d","updated":"2017-12-11 04:10:17.000000000","message":"Ill remove line 26. This can be removed and added later, but the ability to select the endpoints for the exporters will require a label to select them by, and will require the prometheus metrics summary to be added later as well.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"kube-state-metrics/templates/service-kube-metrics.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":22,"context_line":"kind: Service"},{"line_number":23,"context_line":"metadata:"},{"line_number":24,"context_line":"  name: {{ tuple \"kube_metrics\" \"internal\" . | include \"helm-toolkit.endpoints.hostname_short_endpoint_lookup\" }}"},{"line_number":25,"context_line":"  labels:"},{"line_number":26,"context_line":"{{ tuple $envAll \"kube-state-metrics\" \"metrics\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" | indent 4 }}"},{"line_number":27,"context_line":"  annotations:"},{"line_number":28,"context_line":"{{ tuple $endpoint $envAll | include \"helm-toolkit.snippets.prometheus_service_annotations\" | indent 4 }}"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_26f577ed","line":25,"updated":"2017-12-11 02:20:52.000000000","message":"Is this change in scope of this PS?","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":22,"context_line":"kind: Service"},{"line_number":23,"context_line":"metadata:"},{"line_number":24,"context_line":"  name: {{ tuple \"kube_metrics\" \"internal\" . | include \"helm-toolkit.endpoints.hostname_short_endpoint_lookup\" }}"},{"line_number":25,"context_line":"  labels:"},{"line_number":26,"context_line":"{{ tuple $envAll \"kube-state-metrics\" \"metrics\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" | indent 4 }}"},{"line_number":27,"context_line":"  annotations:"},{"line_number":28,"context_line":"{{ tuple $endpoint $envAll | include \"helm-toolkit.snippets.prometheus_service_annotations\" | indent 4 }}"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_66f61f66","line":25,"in_reply_to":"df87a7cf_26f577ed","updated":"2017-12-11 04:10:17.000000000","message":"Same","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"kube-state-metrics/templates/service-scheduler.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  name: kube-scheduler-discovery"},{"line_number":25,"context_line":"  labels:"},{"line_number":26,"context_line":"    component: kube-scheduler"},{"line_number":27,"context_line":"{{ tuple $envAll \"kube-scheduler\" \"metrics\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" | indent 4 }}"},{"line_number":28,"context_line":"  annotations:"},{"line_number":29,"context_line":"{{ tuple $endpoint $envAll  | include \"helm-toolkit.snippets.prometheus_service_annotations\" | indent 4 }}"},{"line_number":30,"context_line":"spec:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_86e5633a","line":27,"updated":"2017-12-11 02:20:52.000000000","message":"this will declare the `component` key twice (https://github.com/openstack/openstack-helm-infra/blob/master/helm-toolkit/templates/snippets/_kubernetes_metadata_labels.tpl#L23). line 26 should probably be removed? Also is this change in scope of this PS?","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  name: kube-scheduler-discovery"},{"line_number":25,"context_line":"  labels:"},{"line_number":26,"context_line":"    component: kube-scheduler"},{"line_number":27,"context_line":"{{ tuple $envAll \"kube-scheduler\" \"metrics\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" | indent 4 }}"},{"line_number":28,"context_line":"  annotations:"},{"line_number":29,"context_line":"{{ tuple $endpoint $envAll  | include \"helm-toolkit.snippets.prometheus_service_annotations\" | indent 4 }}"},{"line_number":30,"context_line":"spec:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_06e9537e","line":27,"in_reply_to":"df87a7cf_86e5633a","updated":"2017-12-11 04:10:17.000000000","message":"Same","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"tools/gate/playbooks/deploy-helm-packages/tasks/util-common-helm-test.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"710b33871ca26dab6ad36da855fb363cb681d4e7","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        - test_result | succeeded"},{"line_number":45,"context_line":"      shell: |-"},{"line_number":46,"context_line":"              set -e"},{"line_number":47,"context_line":"              kubectl logs {{ release }}-test -n {{ namespace }} \u003e\u003e {{ logs_dir }}/helm-tests/{{ release }}.txt"},{"line_number":48,"context_line":"      args:"},{"line_number":49,"context_line":"        executable: /bin/bash"},{"line_number":50,"context_line":"      register: test_logs"}],"source_content_type":"text/x-yaml","patch_set":74,"id":"df87a7cf_2510fa56","line":47,"range":{"start_line":47,"start_character":27,"end_line":47,"end_character":45},"updated":"2017-12-11 18:36:57.000000000","message":"this won\u0027t work for some of the charts in OSH, but I think this is the correct approach, and we should adjust the existing charts to match.","commit_id":"3a8e4ff957d606de6da4945773cf84559f6c6fc2"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"db67fbf61db672340697561d941bb27d0b6ee6d1","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        - test_result | succeeded"},{"line_number":45,"context_line":"      shell: |-"},{"line_number":46,"context_line":"              set -e"},{"line_number":47,"context_line":"              kubectl logs {{ release }}-test -n {{ namespace }} \u003e\u003e {{ logs_dir }}/helm-tests/{{ release }}.txt"},{"line_number":48,"context_line":"      args:"},{"line_number":49,"context_line":"        executable: /bin/bash"},{"line_number":50,"context_line":"      register: test_logs"}],"source_content_type":"text/x-yaml","patch_set":74,"id":"df87a7cf_ebe2e19f","line":47,"range":{"start_line":47,"start_character":27,"end_line":47,"end_character":45},"in_reply_to":"df87a7cf_2510fa56","updated":"2017-12-11 20:10:19.000000000","message":"I agree. I\u0027ll submit a change to rename the helm test pods in openstack-helm to bring them all in line","commit_id":"3a8e4ff957d606de6da4945773cf84559f6c6fc2"}],"tools/gate/playbooks/describe-kubernetes-resources/tasks/main.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":10,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: \"Gathering descriptions for openstack namespace\""},{"line_number":14,"context_line":"  include: util-common-namespace-describe.yaml"},{"line_number":15,"context_line":"  vars:"},{"line_number":16,"context_line":"    api_object: \"{{ openstack_object }}\""},{"line_number":17,"context_line":"    namespace: openstack"},{"line_number":18,"context_line":"  loop_control:"},{"line_number":19,"context_line":"    loop_var: openstack_object"},{"line_number":20,"context_line":"  with_items:"},{"line_number":21,"context_line":"    - pod"},{"line_number":22,"context_line":"    - service"},{"line_number":23,"context_line":"    - pvc"},{"line_number":24,"context_line":"    - deployment"},{"line_number":25,"context_line":"    - statefulset"},{"line_number":26,"context_line":"    - daemonset"},{"line_number":27,"context_line":"    - serviceaccount"},{"line_number":28,"context_line":"    - endpoints"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- name: \"Gathering descriptions for kube-system namespace\""},{"line_number":31,"context_line":"  include: util-common-namespace-describe.yaml"},{"line_number":32,"context_line":"  vars:"},{"line_number":33,"context_line":"    api_object: \"{{ kube_system_object }}\""},{"line_number":34,"context_line":"    namespace: kube-system"},{"line_number":35,"context_line":"  loop_control:"},{"line_number":36,"context_line":"    loop_var: kube_system_object"},{"line_number":37,"context_line":"  with_items:"},{"line_number":38,"context_line":"    - pod"},{"line_number":39,"context_line":"    - service"},{"line_number":40,"context_line":"    - pvc"},{"line_number":41,"context_line":"    - deployment"},{"line_number":42,"context_line":"    - statefulset"},{"line_number":43,"context_line":"    - daemonset"},{"line_number":44,"context_line":"    - serviceaccount"},{"line_number":45,"context_line":"    - endpoints"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- name: \"Gathering descriptions for cluster scoped objects\""},{"line_number":48,"context_line":"  include: util-common-cluster-describe.yaml"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_c6c84ba7","line":45,"range":{"start_line":13,"start_character":0,"end_line":45,"end_character":15},"updated":"2017-12-11 02:20:52.000000000","message":"can we make this an iterable - as we will need to include the ceph and many other namespaces.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"tools/gate/playbooks/describe-kubernetes-resources/tasks/util-common-namespace-describe.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"  block:"},{"line_number":19,"context_line":"    - name: \"creating directory for {{ api_object }} descriptions in {{ namespace }} namespace\""},{"line_number":20,"context_line":"      file: path\u003d\"{{ logs_dir }}/resources/{{ namespace }}/{{ api_object }}\" state\u003ddirectory"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    - name: \"gathering names of {{ api_object }}s currently deployed in {{ namespace }} namespace\""},{"line_number":23,"context_line":"      shell: |-"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_c64dab3e","line":20,"range":{"start_line":20,"start_character":6,"end_line":20,"end_character":92},"updated":"2017-12-11 02:20:52.000000000","message":"for consistency can we use yaml formatted ansible.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"  block:"},{"line_number":19,"context_line":"    - name: \"creating directory for {{ api_object }} descriptions in {{ namespace }} namespace\""},{"line_number":20,"context_line":"      file: path\u003d\"{{ logs_dir }}/resources/{{ namespace }}/{{ api_object }}\" state\u003ddirectory"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    - name: \"gathering names of {{ api_object }}s currently deployed in {{ namespace }} namespace\""},{"line_number":23,"context_line":"      shell: |-"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_e601cf39","line":20,"range":{"start_line":20,"start_character":6,"end_line":20,"end_character":92},"in_reply_to":"df87a7cf_c64dab3e","updated":"2017-12-11 04:10:17.000000000","message":"Done","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"tools/gate/playbooks/gather-pod-logs/tasks/main.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: \"creating directory for pod logs\""},{"line_number":14,"context_line":"  file: path\u003d\"{{ logs_dir }}/pod-logs\" state\u003ddirectory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- include: util-container-logs.yaml"},{"line_number":17,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_e648ef2d","line":14,"range":{"start_line":14,"start_character":2,"end_line":14,"end_character":54},"updated":"2017-12-11 02:20:52.000000000","message":"for consistency can we use yaml formated ansible.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: \"creating directory for pod logs\""},{"line_number":14,"context_line":"  file: path\u003d\"{{ logs_dir }}/pod-logs\" state\u003ddirectory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- include: util-container-logs.yaml"},{"line_number":17,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_46c19be4","line":14,"range":{"start_line":14,"start_character":2,"end_line":14,"end_character":54},"in_reply_to":"df87a7cf_e648ef2d","updated":"2017-12-11 04:10:17.000000000","message":"Done","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":13,"context_line":"- name: \"creating directory for pod logs\""},{"line_number":14,"context_line":"  file: path\u003d\"{{ logs_dir }}/pod-logs\" state\u003ddirectory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- include: util-container-logs.yaml"},{"line_number":17,"context_line":"  vars:"},{"line_number":18,"context_line":"    namespace: kube-system"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"- include: util-container-logs.yaml"},{"line_number":21,"context_line":"  vars:"},{"line_number":22,"context_line":"    namespace: openstack"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: \"Downloads logs to executor\""},{"line_number":25,"context_line":"  synchronize:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_a64e6735","line":22,"range":{"start_line":16,"start_character":0,"end_line":22,"end_character":24},"updated":"2017-12-11 02:20:52.000000000","message":"can we iterate over this - as we will need to accommodate many more namespaces than just kube-system and OpenStack. Ideally, we could query k8s for all namespaces present.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":13,"context_line":"- name: \"creating directory for pod logs\""},{"line_number":14,"context_line":"  file: path\u003d\"{{ logs_dir }}/pod-logs\" state\u003ddirectory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- include: util-container-logs.yaml"},{"line_number":17,"context_line":"  vars:"},{"line_number":18,"context_line":"    namespace: kube-system"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"- include: util-container-logs.yaml"},{"line_number":21,"context_line":"  vars:"},{"line_number":22,"context_line":"    namespace: openstack"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: \"Downloads logs to executor\""},{"line_number":25,"context_line":"  synchronize:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_66c4dff5","line":22,"range":{"start_line":16,"start_character":0,"end_line":22,"end_character":24},"in_reply_to":"df87a7cf_a64e6735","updated":"2017-12-11 04:10:17.000000000","message":"Done","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"tools/gate/playbooks/gather-prom-metrics/tasks/main.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: \"creating directory for helm release descriptions\""},{"line_number":14,"context_line":"  file: path\u003d\"{{ logs_dir }}/prometheus\" state\u003ddirectory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: \"get exporter services in kube-system namespace\""},{"line_number":17,"context_line":"  shell: |-"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_a61507e7","line":14,"range":{"start_line":14,"start_character":1,"end_line":14,"end_character":56},"updated":"2017-12-11 02:20:52.000000000","message":"can we use yaml syntax for consistency.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: \"creating directory for helm release descriptions\""},{"line_number":14,"context_line":"  file: path\u003d\"{{ logs_dir }}/prometheus\" state\u003ddirectory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: \"get exporter services in kube-system namespace\""},{"line_number":17,"context_line":"  shell: |-"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_26ba576d","line":14,"range":{"start_line":14,"start_character":1,"end_line":14,"end_character":56},"in_reply_to":"df87a7cf_a61507e7","updated":"2017-12-11 04:10:17.000000000","message":"Done","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"tools/gate/playbooks/helm-release-status/tasks/main.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":10,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: \"creating directory for helm release descriptions\""},{"line_number":14,"context_line":"  file: path\u003d\"{{ logs_dir }}/helm-releases\" state\u003ddirectory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: \"creating directory for helm test logs\""},{"line_number":17,"context_line":"  file: path\u003d\"{{ logs_dir }}/helm-tests\" state\u003ddirectory"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"- name: \"Gathering release status and helm test logs in chart groups\""},{"line_number":20,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_06dcf343","line":17,"range":{"start_line":13,"start_character":0,"end_line":17,"end_character":56},"updated":"2017-12-11 02:20:52.000000000","message":"these can be combined into a single command.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":10,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: \"creating directory for helm release descriptions\""},{"line_number":14,"context_line":"  file: path\u003d\"{{ logs_dir }}/helm-releases\" state\u003ddirectory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: \"creating directory for helm test logs\""},{"line_number":17,"context_line":"  file: path\u003d\"{{ logs_dir }}/helm-tests\" state\u003ddirectory"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"- name: \"Gathering release status and helm test logs in chart groups\""},{"line_number":20,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_e6af8f27","line":17,"range":{"start_line":13,"start_character":0,"end_line":17,"end_character":56},"in_reply_to":"df87a7cf_06dcf343","updated":"2017-12-11 04:10:17.000000000","message":"Done","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"tools/gate/playbooks/helm-release-status/tasks/util-common-release-test.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"6435dd951b25e838e8c00b2ecec1003bcccbcb46","unresolved":false,"context_lines":[{"line_number":17,"context_line":"  block:"},{"line_number":18,"context_line":"    - name: \"check if release {{ release }} tests executed\""},{"line_number":19,"context_line":"      shell: |-"},{"line_number":20,"context_line":"              helm status \"{{ release }}\" | grep test | awk \u0027{print $1}\u0027"},{"line_number":21,"context_line":"      args:"},{"line_number":22,"context_line":"        executable: /bin/bash"},{"line_number":23,"context_line":"      register: release_test"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_c697eb47","line":20,"range":{"start_line":20,"start_character":44,"end_line":20,"end_character":53},"updated":"2017-12-11 02:20:52.000000000","message":"this needs to be made much tighter - as it will hit a +ve result for any line with \u0027test\u0027 in. Would it perhaps make sense to output the logs from the test container as it was with the old gate script during runtime (https://github.com/openstack/openstack-helm/blob/master/tools/gate/funcs/helm.sh#L79-L99), and this task can simply be reduced to collating them? This approach would also allow us to remove the test container, allowing helm test to be run multiple times.","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8316430a4aa233c02d98a75532b844c566061e85","unresolved":false,"context_lines":[{"line_number":17,"context_line":"  block:"},{"line_number":18,"context_line":"    - name: \"check if release {{ release }} tests executed\""},{"line_number":19,"context_line":"      shell: |-"},{"line_number":20,"context_line":"              helm status \"{{ release }}\" | grep test | awk \u0027{print $1}\u0027"},{"line_number":21,"context_line":"      args:"},{"line_number":22,"context_line":"        executable: /bin/bash"},{"line_number":23,"context_line":"      register: release_test"}],"source_content_type":"text/x-yaml","patch_set":72,"id":"df87a7cf_0645d36f","line":20,"range":{"start_line":20,"start_character":44,"end_line":20,"end_character":53},"in_reply_to":"df87a7cf_c697eb47","updated":"2017-12-11 04:10:17.000000000","message":"The current ansible tasks for running helm test include flags defined in the chart_deploys for outputting the helm test logs during runtime, as requested when that functionality was added. That can be modified to do what you\u0027re requesting","commit_id":"3ca2c89fa33533c25de57e35443127eb69a29d96"}],"tools/gate/playbooks/post-run-info/tasks/util-gather-pod-logs.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"34c547ac956ede6b2f0f1ae9f741d863f7101c29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"df87a7cf_3e9db2bb","updated":"2017-12-07 19:59:59.000000000","message":"Though this shows that the fluentbit pods are running - I don\u0027t see what this gets us over just copying the container logs directly from /var/log/containers? The only thing this seems to validate that the bind mount is operating as expected?","commit_id":"3119e99232de80fb839333da6ae4508bf7f15c8c"}],"tools/gate/playbooks/vars.yaml":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"7f4fe3f9816a8dc5a2e57b3160dce60bc397670e","unresolved":false,"context_lines":[{"line_number":28,"context_line":"  cluster:"},{"line_number":29,"context_line":"    cni: calico"},{"line_number":30,"context_line":"    pod_subnet: 192.168.0.0/16"},{"line_number":31,"context_line":"  namespaced_objects:"},{"line_number":32,"context_line":"    - pod"},{"line_number":33,"context_line":"    - service"},{"line_number":34,"context_line":"    - pvc"},{"line_number":35,"context_line":"    - deployment"},{"line_number":36,"context_line":"    - statefulset"},{"line_number":37,"context_line":"    - daemonset"},{"line_number":38,"context_line":"    - serviceaccount"},{"line_number":39,"context_line":"  cluster_objects:"},{"line_number":40,"context_line":"    - node"},{"line_number":41,"context_line":"    - clusterrole"},{"line_number":42,"context_line":"    - clusterrolebinding"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"nodes:"},{"line_number":45,"context_line":"  labels:"}],"source_content_type":"text/x-yaml","patch_set":21,"id":"df87a7cf_bbf6d0f8","line":42,"range":{"start_line":31,"start_character":0,"end_line":42,"end_character":24},"updated":"2017-12-06 21:47:08.000000000","message":"do we want these to be configurable?","commit_id":"6c5f3e2851417f1b5b7a9a9327e65f5ef09c888c"}]}