)]}'
{"tools/deployment/apparmor/020-ceph.sh":[{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"9bd30be37e2886e28f20eb05ed3b83eae75f35c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":40,"id":"3fa7e38b_edfaf241","updated":"2020-02-12 22:11:06.000000000","message":"I don\u0027t think this is needed","commit_id":"d49c103babdd00c89c3c2d3af93d3d52318c6565"},{"author":{"_account_id":29131,"name":"diwakar thyagaraj","email":"diwakar.chitoor.thyagaraj@att.com","username":"dt241s@att.com"},"change_message_id":"ccb1ae435c08a4fb23ad9eb7c342f0a76038082c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":40,"id":"3fa7e38b_ee2c29b3","in_reply_to":"3fa7e38b_edfaf241","updated":"2020-02-13 00:51:03.000000000","message":"We have app armor for ceph as well so needed to test that as well.","commit_id":"d49c103babdd00c89c3c2d3af93d3d52318c6565"},{"author":{"_account_id":30777,"name":"Steven Fitzpatrick","email":"steven@fitzpatrick.wtf","username":"sf280x"},"change_message_id":"f79a71262101ab89b08ef25c84e249fdfe42b61b","unresolved":false,"context_lines":[{"line_number":17,"context_line":"set -xe"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"#NOTE: Lint and package chart"},{"line_number":20,"context_line":"for CHART in ceph-mon ceph-client ceph-provisioners; do"},{"line_number":21,"context_line":"  make \"${CHART}\""},{"line_number":22,"context_line":"done"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-sh","patch_set":54,"id":"3fa7e38b_ca3580a8","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":55},"updated":"2020-02-14 08:23:54.000000000","message":"I was comparing this and the ../osh-infra-logging/020-ceph.sh scripts. A big difference I can see is here the ceph-osd chart is not made or installed. IIRC, it\u0027s the osds which provide the actual storage in ceph, so I think this could be why Prometheus is failing in Zuul.","commit_id":"8b7893cf9843bb575e5a7c68a6b61f50b64d280f"},{"author":{"_account_id":30777,"name":"Steven Fitzpatrick","email":"steven@fitzpatrick.wtf","username":"sf280x"},"change_message_id":"f79a71262101ab89b08ef25c84e249fdfe42b61b","unresolved":false,"context_lines":[{"line_number":197,"context_line":"      ceph-osd-default: runtime/default"},{"line_number":198,"context_line":"EOF"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"for CHART in ceph-mon ceph-client ceph-provisioners; do"},{"line_number":201,"context_line":"  helm upgrade --install ${CHART} ./${CHART} \\"},{"line_number":202,"context_line":"    --namespace\u003dceph \\"},{"line_number":203,"context_line":"    --values\u003d/tmp/ceph.yaml \\"}],"source_content_type":"text/x-sh","patch_set":54,"id":"3fa7e38b_8a3b0898","line":200,"range":{"start_line":200,"start_character":0,"end_line":200,"end_character":55},"updated":"2020-02-14 08:23:54.000000000","message":"Don\u0027t forget to add it to this loop too.","commit_id":"8b7893cf9843bb575e5a7c68a6b61f50b64d280f"},{"author":{"_account_id":30777,"name":"Steven Fitzpatrick","email":"steven@fitzpatrick.wtf","username":"sf280x"},"change_message_id":"97a9da57f14c8898aa16c5562abdb5bc959cd2df","unresolved":false,"context_lines":[{"line_number":205,"context_line":"    ${OSH_INFRA_EXTRA_HELM_ARGS_CEPH_DEPLOY:-$(./tools/deployment/common/get-values-overrides.sh ${CHART})}"},{"line_number":206,"context_line":"done"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"helm upgrade --install ceph-osd ./ceph-osd \\"},{"line_number":209,"context_line":"  --namespace\u003dceph \\"},{"line_number":210,"context_line":"  --values\u003d/tmp/ceph.yaml \\"},{"line_number":211,"context_line":"  --values\u003d/tmp/ceph-osd.yaml"}],"source_content_type":"text/x-sh","patch_set":54,"id":"3fa7e38b_2a1d340f","line":208,"range":{"start_line":208,"start_character":0,"end_line":208,"end_character":44},"updated":"2020-02-14 08:31:20.000000000","message":"Sorry, I missed this obviously...","commit_id":"8b7893cf9843bb575e5a7c68a6b61f50b64d280f"}],"tools/deployment/osh-infra-monitoring/050-prometheus.sh":[{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"9bd30be37e2886e28f20eb05ed3b83eae75f35c4","unresolved":false,"context_lines":[{"line_number":19,"context_line":"#NOTE: Lint and package chart"},{"line_number":20,"context_line":"make prometheus"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"FEATURE_GATES\u003d\"alertmanager,ceph,elasticsearch,kubernetes,nodes,openstack,postgresql\""},{"line_number":23,"context_line":": ${OSH_INFRA_EXTRA_HELM_ARGS_PROMETHEUS:\u003d\"$({ ./tools/deployment/common/get-values-overrides.sh prometheus;} 2\u003e /dev/null)\"}"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"#NOTE: Deploy command"},{"line_number":26,"context_line":"helm upgrade --install prometheus ./prometheus \\"}],"source_content_type":"text/x-sh","patch_set":40,"id":"3fa7e38b_ed7f52b9","side":"PARENT","line":23,"range":{"start_line":22,"start_character":0,"end_line":23,"end_character":125},"updated":"2020-02-12 22:11:06.000000000","message":"why was this removed?","commit_id":"b21fdfabada05c68d9c7a922a4259725310030bf"},{"author":{"_account_id":29131,"name":"diwakar thyagaraj","email":"diwakar.chitoor.thyagaraj@att.com","username":"dt241s@att.com"},"change_message_id":"ccb1ae435c08a4fb23ad9eb7c342f0a76038082c","unresolved":false,"context_lines":[{"line_number":19,"context_line":"#NOTE: Lint and package chart"},{"line_number":20,"context_line":"make prometheus"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"FEATURE_GATES\u003d\"alertmanager,ceph,elasticsearch,kubernetes,nodes,openstack,postgresql\""},{"line_number":23,"context_line":": ${OSH_INFRA_EXTRA_HELM_ARGS_PROMETHEUS:\u003d\"$({ ./tools/deployment/common/get-values-overrides.sh prometheus;} 2\u003e /dev/null)\"}"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"#NOTE: Deploy command"},{"line_number":26,"context_line":"helm upgrade --install prometheus ./prometheus \\"}],"source_content_type":"text/x-sh","patch_set":40,"id":"3fa7e38b_ae32b10c","side":"PARENT","line":23,"range":{"start_line":22,"start_character":0,"end_line":23,"end_character":125},"in_reply_to":"3fa7e38b_ed7f52b9","updated":"2020-02-13 00:51:03.000000000","message":"Reverted","commit_id":"b21fdfabada05c68d9c7a922a4259725310030bf"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"43b03a10ab11bac25c67a43937fdad91939e06c7","unresolved":false,"context_lines":[{"line_number":35,"context_line":"helm status prometheus"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"helm test prometheus"},{"line_number":38,"context_line":"kubectl describe pod prometheus-prometheus-0 -n osh-infra"}],"source_content_type":"text/x-sh","patch_set":57,"id":"3fa7e38b_db41c938","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":57},"updated":"2020-02-14 19:17:06.000000000","message":"This isn\u0027t really needed","commit_id":"95a3baeadfa121039c824769649495bce4d41903"}],"zuul.d/jobs.yaml":[{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"9bd30be37e2886e28f20eb05ed3b83eae75f35c4","unresolved":false,"context_lines":[{"line_number":272,"context_line":"        - ./tools/deployment/apparmor/005-deploy-k8s.sh"},{"line_number":273,"context_line":"        - ./tools/deployment/apparmor/020-ceph.sh"},{"line_number":274,"context_line":"        - ./tools/deployment/apparmor/040-memcached.sh"},{"line_number":275,"context_line":"        - ./tools/deployment/apparmor/050-prometheus-alertmanager.sh"},{"line_number":276,"context_line":"        - ./tools/deployment/apparmor/060-prometheus-node-exporter.sh"},{"line_number":277,"context_line":"        - ./tools/deployment/apparmor/070-prometheus-openstack-exporter.sh"},{"line_number":278,"context_line":"        - ./tools/deployment/apparmor/080-prometheus-process-exporter.sh"}],"source_content_type":"text/x-yaml","patch_set":40,"id":"3fa7e38b_6dc60218","side":"PARENT","line":275,"range":{"start_line":275,"start_character":38,"end_line":275,"end_character":68},"updated":"2020-02-12 22:11:06.000000000","message":"Why was this removed?","commit_id":"b21fdfabada05c68d9c7a922a4259725310030bf"},{"author":{"_account_id":29131,"name":"diwakar thyagaraj","email":"diwakar.chitoor.thyagaraj@att.com","username":"dt241s@att.com"},"change_message_id":"ccb1ae435c08a4fb23ad9eb7c342f0a76038082c","unresolved":false,"context_lines":[{"line_number":272,"context_line":"        - ./tools/deployment/apparmor/005-deploy-k8s.sh"},{"line_number":273,"context_line":"        - ./tools/deployment/apparmor/020-ceph.sh"},{"line_number":274,"context_line":"        - ./tools/deployment/apparmor/040-memcached.sh"},{"line_number":275,"context_line":"        - ./tools/deployment/apparmor/050-prometheus-alertmanager.sh"},{"line_number":276,"context_line":"        - ./tools/deployment/apparmor/060-prometheus-node-exporter.sh"},{"line_number":277,"context_line":"        - ./tools/deployment/apparmor/070-prometheus-openstack-exporter.sh"},{"line_number":278,"context_line":"        - ./tools/deployment/apparmor/080-prometheus-process-exporter.sh"}],"source_content_type":"text/x-yaml","patch_set":40,"id":"3fa7e38b_8e1b3582","side":"PARENT","line":275,"range":{"start_line":275,"start_character":38,"end_line":275,"end_character":68},"in_reply_to":"3fa7e38b_6dc60218","updated":"2020-02-13 00:51:03.000000000","message":"revereted","commit_id":"b21fdfabada05c68d9c7a922a4259725310030bf"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"9bd30be37e2886e28f20eb05ed3b83eae75f35c4","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        - ./tools/deployment/apparmor/001-setup-apparmor-profiles.sh"},{"line_number":276,"context_line":"        - ./tools/deployment/apparmor/005-deploy-k8s.sh"},{"line_number":277,"context_line":"        - ./tools/deployment/apparmor/020-ceph.sh"},{"line_number":278,"context_line":"        - ./tools/deployment/apparmor/030-nfs-provisioner.sh"},{"line_number":279,"context_line":"        - ./tools/deployment/apparmor/040-memcached.sh"},{"line_number":280,"context_line":"        - ./tools/deployment/apparmor/055-prometheus.sh"},{"line_number":281,"context_line":"        - ./tools/deployment/apparmor/060-prometheus-node-exporter.sh"}],"source_content_type":"text/x-yaml","patch_set":40,"id":"3fa7e38b_cdd616c5","line":278,"range":{"start_line":278,"start_character":10,"end_line":278,"end_character":60},"updated":"2020-02-12 22:11:06.000000000","message":"I don\u0027t think prometheus needs nfs, it should run fine with ceph as the storage backend.\n\nSomeone who is more familiar with prometheus can probably speak better to this than I can though.","commit_id":"d49c103babdd00c89c3c2d3af93d3d52318c6565"},{"author":{"_account_id":29131,"name":"diwakar thyagaraj","email":"diwakar.chitoor.thyagaraj@att.com","username":"dt241s@att.com"},"change_message_id":"ccb1ae435c08a4fb23ad9eb7c342f0a76038082c","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        - ./tools/deployment/apparmor/001-setup-apparmor-profiles.sh"},{"line_number":276,"context_line":"        - ./tools/deployment/apparmor/005-deploy-k8s.sh"},{"line_number":277,"context_line":"        - ./tools/deployment/apparmor/020-ceph.sh"},{"line_number":278,"context_line":"        - ./tools/deployment/apparmor/030-nfs-provisioner.sh"},{"line_number":279,"context_line":"        - ./tools/deployment/apparmor/040-memcached.sh"},{"line_number":280,"context_line":"        - ./tools/deployment/apparmor/055-prometheus.sh"},{"line_number":281,"context_line":"        - ./tools/deployment/apparmor/060-prometheus-node-exporter.sh"}],"source_content_type":"text/x-yaml","patch_set":40,"id":"3fa7e38b_4e21bdb1","line":278,"range":{"start_line":278,"start_character":10,"end_line":278,"end_character":60},"in_reply_to":"3fa7e38b_cdd616c5","updated":"2020-02-13 00:51:03.000000000","message":"Yes storage was not getting picked up, hence added in value overrides and also openstack control panel was not enabled, Enabled it back as well.","commit_id":"d49c103babdd00c89c3c2d3af93d3d52318c6565"}],"zuul.d/project.yaml":[{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"9bd30be37e2886e28f20eb05ed3b83eae75f35c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":40,"id":"3fa7e38b_2de56aa3","updated":"2020-02-12 22:11:06.000000000","message":"obv revert this file when its no longer WIP","commit_id":"d49c103babdd00c89c3c2d3af93d3d52318c6565"},{"author":{"_account_id":29131,"name":"diwakar thyagaraj","email":"diwakar.chitoor.thyagaraj@att.com","username":"dt241s@att.com"},"change_message_id":"ccb1ae435c08a4fb23ad9eb7c342f0a76038082c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":40,"id":"3fa7e38b_ae0b912a","in_reply_to":"3fa7e38b_2de56aa3","updated":"2020-02-13 00:51:03.000000000","message":"Sure Will do it.","commit_id":"d49c103babdd00c89c3c2d3af93d3d52318c6565"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"72607656e565cdace3ce91f854a13ddb4a2b0260","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- project:"},{"line_number":17,"context_line":"    templates:"},{"line_number":18,"context_line":"      - publish-openstack-docs-pti"},{"line_number":19,"context_line":"    check:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - openstack-helm-infra-apparmor:"},{"line_number":22,"context_line":"    gate:"},{"line_number":23,"context_line":"      jobs:"},{"line_number":24,"context_line":"        - openstack-helm-lint"},{"line_number":25,"context_line":"        - openstack-helm-infra-aio-logging"},{"line_number":26,"context_line":"        - openstack-helm-infra-aio-monitoring"},{"line_number":27,"context_line":"        - openstack-helm-infra-openstack-support"},{"line_number":28,"context_line":"    # NOTE(srwilkers): Disabling all periodic and experimental jobs until"},{"line_number":29,"context_line":"    # issues with the kubeadm-aio based deployments are addressed"},{"line_number":30,"context_line":"    # periodic:"},{"line_number":31,"context_line":"    #   jobs:"},{"line_number":32,"context_line":"        # - openstack-helm-infra-tenant-ceph"},{"line_number":33,"context_line":"        # - openstack-helm-infra-five-ubuntu"},{"line_number":34,"context_line":"        # - openstack-helm-infra-armada-deploy"},{"line_number":35,"context_line":"        # - openstack-helm-infra-armada-update-uuid"},{"line_number":36,"context_line":"        # - openstack-helm-infra-armada-update-passwords"},{"line_number":37,"context_line":"    experimental:"},{"line_number":38,"context_line":"      jobs:"},{"line_number":39,"context_line":"        # NOTE(srwilkers): Disable fedora experimental jobs until issues resolved"},{"line_number":40,"context_line":"        #- openstack-helm-infra-five-fedora"},{"line_number":41,"context_line":"        # NOTE(srwilkers): Disable centos experimental jobs until issues resolved"},{"line_number":42,"context_line":"        #- openstack-helm-infra-five-centos"},{"line_number":43,"context_line":"        # - openstack-helm-infra-five-ubuntu"},{"line_number":44,"context_line":"        - openstack-helm-infra-elastic-beats"},{"line_number":45,"context_line":"        # - openstack-helm-infra-tenant-ceph"},{"line_number":46,"context_line":"        # - openstack-helm-infra-armada-deploy"},{"line_number":47,"context_line":"        # - openstack-helm-infra-armada-update-uuid"},{"line_number":48,"context_line":"        # - openstack-helm-infra-armada-update-passwords"}],"source_content_type":"text/x-yaml","patch_set":55,"id":"3fa7e38b_2c9eac54","line":48,"range":{"start_line":16,"start_character":2,"end_line":48,"end_character":0},"updated":"2020-02-14 15:17:50.000000000","message":"expected str for dictionary value @ data[\u0027check\u0027][\u0027jobs\u0027][0][\u0027openstack-helm-infra-apparmor\u0027]","commit_id":"56c6798509b953fd1d1967dd30f9dcce7f5db92c"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"50f613bf03f283624ebce996f1d3217544c84f0f","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- project:"},{"line_number":17,"context_line":"    templates:"},{"line_number":18,"context_line":"      - publish-openstack-docs-pti"},{"line_number":19,"context_line":"    check:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - openstack-helm-infra-apparmor:"},{"line_number":22,"context_line":"    gate:"},{"line_number":23,"context_line":"      jobs:"},{"line_number":24,"context_line":"        - openstack-helm-lint"},{"line_number":25,"context_line":"        - openstack-helm-infra-aio-logging"},{"line_number":26,"context_line":"        - openstack-helm-infra-aio-monitoring"},{"line_number":27,"context_line":"        - openstack-helm-infra-openstack-support"},{"line_number":28,"context_line":"    # NOTE(srwilkers): Disabling all periodic and experimental jobs until"},{"line_number":29,"context_line":"    # issues with the kubeadm-aio based deployments are addressed"},{"line_number":30,"context_line":"    # periodic:"},{"line_number":31,"context_line":"    #   jobs:"},{"line_number":32,"context_line":"        # - openstack-helm-infra-tenant-ceph"},{"line_number":33,"context_line":"        # - openstack-helm-infra-five-ubuntu"},{"line_number":34,"context_line":"        # - openstack-helm-infra-armada-deploy"},{"line_number":35,"context_line":"        # - openstack-helm-infra-armada-update-uuid"},{"line_number":36,"context_line":"        # - openstack-helm-infra-armada-update-passwords"},{"line_number":37,"context_line":"    experimental:"},{"line_number":38,"context_line":"      jobs:"},{"line_number":39,"context_line":"        # NOTE(srwilkers): Disable fedora experimental jobs until issues resolved"},{"line_number":40,"context_line":"        #- openstack-helm-infra-five-fedora"},{"line_number":41,"context_line":"        # NOTE(srwilkers): Disable centos experimental jobs until issues resolved"},{"line_number":42,"context_line":"        #- openstack-helm-infra-five-centos"},{"line_number":43,"context_line":"        # - openstack-helm-infra-five-ubuntu"},{"line_number":44,"context_line":"        - openstack-helm-infra-elastic-beats"},{"line_number":45,"context_line":"        # - openstack-helm-infra-tenant-ceph"},{"line_number":46,"context_line":"        # - openstack-helm-infra-armada-deploy"},{"line_number":47,"context_line":"        # - openstack-helm-infra-armada-update-uuid"},{"line_number":48,"context_line":"        # - openstack-helm-infra-armada-update-passwords"}],"source_content_type":"text/x-yaml","patch_set":62,"id":"3fa7e38b_3b357d43","line":48,"range":{"start_line":16,"start_character":2,"end_line":48,"end_character":0},"updated":"2020-02-14 20:38:12.000000000","message":"expected str for dictionary value @ data[\u0027check\u0027][\u0027jobs\u0027][0][\u0027openstack-helm-infra-apparmor\u0027]","commit_id":"004db3493354b7f545531ab353839a0b6402307c"}]}