)]}'
{"jenkins/jobs/kolla.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6d71491cd9416dc93054f02ad85d69f5498a2df6","unresolved":false,"context_lines":[{"line_number":58,"context_line":"      - console-log"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job-template:"},{"line_number":61,"context_line":"    name: \u0027publish-kolla-dsvm-build-{distro}-{type}-{node}{suffix}\u0027"},{"line_number":62,"context_line":"    node: \u0027{node}\u0027"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"7f231b9d_30bab509","line":61,"range":{"start_line":61,"start_character":10,"end_line":61,"end_character":67},"updated":"2017-06-14 17:32:13.000000000","message":"I may be missing it but I don\u0027t see this job template being used anywhere.","commit_id":"95d2c02e84b8da5ff4940745e817423cdd212fb3"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"4308190ca9b42ddb3fe6b2afcc4e2e09c829ae21","unresolved":false,"context_lines":[{"line_number":58,"context_line":"      - console-log"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job-template:"},{"line_number":61,"context_line":"    name: \u0027publish-kolla-dsvm-build-{distro}-{type}-{node}{suffix}\u0027"},{"line_number":62,"context_line":"    node: \u0027{node}\u0027"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"7f231b9d_b02a4529","line":61,"range":{"start_line":61,"start_character":10,"end_line":61,"end_character":67},"in_reply_to":"7f231b9d_30bab509","updated":"2017-06-14 17:35:30.000000000","message":"publish-kolla-dsvm-build-ubuntu-source-ubuntu-xenial \u003c- it\u0027s used there, first job in experimental kolla pipe. One that will build images to be tested and publish them to images/tmp","commit_id":"95d2c02e84b8da5ff4940745e817423cdd212fb3"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3fe6917afc3e3b73b0654e8ae085e1c812e29f86","unresolved":false,"context_lines":[{"line_number":58,"context_line":"      - console-log"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job-template:"},{"line_number":61,"context_line":"    name: \u0027publish-kolla-dsvm-build-{distro}-{type}-{node}{suffix}\u0027"},{"line_number":62,"context_line":"    node: \u0027{node}\u0027"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"7f231b9d_43ee89ed","line":61,"range":{"start_line":61,"start_character":10,"end_line":61,"end_character":67},"in_reply_to":"7f231b9d_b02a4529","updated":"2017-06-14 18:15:52.000000000","message":"Right but nothing in JJB instantiates the template. So it won\u0027t exist as a job in the job executors. At least as far as I can tell running grep over JJB configs.","commit_id":"95d2c02e84b8da5ff4940745e817423cdd212fb3"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9ca87b5385708c0b9d677ab1e88ea1392665fce3","unresolved":false,"context_lines":[{"line_number":6288,"context_line":"            - oraclelinux"},{"line_number":6289,"context_line":"          node: centos-7"},{"line_number":6290,"context_line":"          pipeline:"},{"line_number":6291,"context_line":"            - gate"},{"line_number":6292,"context_line":"          type:"},{"line_number":6293,"context_line":"            - binary"},{"line_number":6294,"context_line":"            - source"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"7f231b9d_5da57603","line":6291,"updated":"2017-06-13 15:37:17.000000000","message":"IIUC single item lists are bad, why do you change this?","commit_id":"84c1d835ac68779d81ab979f81a988f4157ce482"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9ca87b5385708c0b9d677ab1e88ea1392665fce3","unresolved":false,"context_lines":[{"line_number":6317,"context_line":"            - kubernetes"},{"line_number":6318,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":6319,"context_line":"          pipeline:"},{"line_number":6320,"context_line":"            - publish"},{"line_number":6321,"context_line":"          type:"},{"line_number":6322,"context_line":"            - binary"},{"line_number":6323,"context_line":"            - source"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"7f231b9d_9d842e60","line":6320,"updated":"2017-06-13 15:37:17.000000000","message":"Also single item, more below","commit_id":"84c1d835ac68779d81ab979f81a988f4157ce482"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"43d0354cfcf49332b00be211bc2bb3d67608021f","unresolved":false,"context_lines":[{"line_number":6298,"context_line":"            - binary"},{"line_number":6299,"context_line":"            - source"},{"line_number":6300,"context_line":"          suffix: \u0027\u0027"},{"line_number":6301,"context_line":"      - \u0027publish-kolla-dsvm-{action}-{distro}-{type}-{node}{suffix}\u0027:"},{"line_number":6302,"context_line":"          action:"},{"line_number":6303,"context_line":"            - build"},{"line_number":6304,"context_line":"            - save"}],"source_content_type":"text/x-yaml","patch_set":19,"id":"7f231b9d_26ba7b09","line":6301,"updated":"2017-06-14 18:43:06.000000000","message":"This template doesn\u0027t have an \u0027action\u0027 parameter. It is just -build-.","commit_id":"04388528c001944c9f39cc096838bcf0e3c8bf26"}],"zuul/layout.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"d518a4e42bb37a411dbf179c8795d4acf0969179","unresolved":false,"context_lines":[{"line_number":11371,"context_line":"      - gate-kolla-dsvm-build-ubuntu-source-ubuntu-xenial"},{"line_number":11372,"context_line":"      - gate-kolla-dsvm-build-oraclelinux-source-centos-7"},{"line_number":11373,"context_line":"    experimental:"},{"line_number":11374,"context_line":"      - prepublish-kolla-dsvm-build-ubuntu-source-ubuntu-xenial"},{"line_number":11375,"context_line":"    post:"},{"line_number":11376,"context_line":"      - kolla-dsvm-build-centos-binary-centos-7"},{"line_number":11377,"context_line":"      - kolla-dsvm-build-centos-source-centos-7"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df140735_104e0cc1","line":11374,"updated":"2017-06-07 06:00:55.000000000","message":"we\u0027ve never had something called prepublish? ... i think this should just be \"gate\"","commit_id":"7ba71e3cb07aa3fcdfc3eee37adc01663fa42494"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"c08c9f1f5ffe1191e3f356aca6e28860543711d9","unresolved":false,"context_lines":[{"line_number":11371,"context_line":"      - gate-kolla-dsvm-build-ubuntu-source-ubuntu-xenial"},{"line_number":11372,"context_line":"      - gate-kolla-dsvm-build-oraclelinux-source-centos-7"},{"line_number":11373,"context_line":"    experimental:"},{"line_number":11374,"context_line":"      - prepublish-kolla-dsvm-build-ubuntu-source-ubuntu-xenial"},{"line_number":11375,"context_line":"    post:"},{"line_number":11376,"context_line":"      - kolla-dsvm-build-centos-binary-centos-7"},{"line_number":11377,"context_line":"      - kolla-dsvm-build-centos-source-centos-7"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df140735_6a909f96","line":11374,"in_reply_to":"df140735_104e0cc1","updated":"2017-06-07 15:00:33.000000000","message":"well, it\u0027s going to be different pipeline than gate, it\u0027s going to test out docker images before publishing them...hence prepublish","commit_id":"7ba71e3cb07aa3fcdfc3eee37adc01663fa42494"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9ca87b5385708c0b9d677ab1e88ea1392665fce3","unresolved":false,"context_lines":[{"line_number":11376,"context_line":"      - publish-kolla-dsvm-build-ubuntu-source-ubuntu-xenial:"},{"line_number":11377,"context_line":"        - publish-kolla-dsvm-ansible-deploy-multinode-ubuntu-source-ubuntu-xenial-2-node:"},{"line_number":11378,"context_line":"          - publish-kolla-dsvm-kubernetes-deploy-multinode-ubuntu-source-ubuntu-xenial-2-node:"},{"line_number":11379,"context_line":"            - publish-kolla-dsvm-save-ubuntu-source-ubuntu-xenial"},{"line_number":11380,"context_line":"    post:"},{"line_number":11381,"context_line":"      - kolla-dsvm-build-centos-binary-centos-7"},{"line_number":11382,"context_line":"      - kolla-dsvm-build-centos-source-centos-7"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"7f231b9d_dde62645","line":11379,"updated":"2017-06-13 15:37:17.000000000","message":"Do these increasing indentations have a special meaning? Or should this just be at the same level and without trailing colon?","commit_id":"84c1d835ac68779d81ab979f81a988f4157ce482"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"a108c77d7bce92f9bae33d50346e02afdd5c8f45","unresolved":false,"context_lines":[{"line_number":11376,"context_line":"      - publish-kolla-dsvm-build-ubuntu-source-ubuntu-xenial:"},{"line_number":11377,"context_line":"        - publish-kolla-dsvm-ansible-deploy-multinode-ubuntu-source-ubuntu-xenial-2-node:"},{"line_number":11378,"context_line":"          - publish-kolla-dsvm-kubernetes-deploy-multinode-ubuntu-source-ubuntu-xenial-2-node:"},{"line_number":11379,"context_line":"            - publish-kolla-dsvm-save-ubuntu-source-ubuntu-xenial"},{"line_number":11380,"context_line":"    post:"},{"line_number":11381,"context_line":"      - kolla-dsvm-build-centos-binary-centos-7"},{"line_number":11382,"context_line":"      - kolla-dsvm-build-centos-source-centos-7"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"7f231b9d_2584dcd3","line":11379,"in_reply_to":"7f231b9d_dde62645","updated":"2017-06-13 17:45:11.000000000","message":"They do, they mean \"run one job after another if previous one succeed\"","commit_id":"84c1d835ac68779d81ab979f81a988f4157ce482"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6d71491cd9416dc93054f02ad85d69f5498a2df6","unresolved":false,"context_lines":[{"line_number":11397,"context_line":"      - name: release-notes-jobs"},{"line_number":11398,"context_line":"      - name: deploy-guide-jobs"},{"line_number":11399,"context_line":"    check:"},{"line_number":11400,"context_line":"      - gate-kolla-ansible-dsvm-deploy-centos-binary-centos-7-nv"},{"line_number":11401,"context_line":"      - gate-kolla-ansible-dsvm-deploy-centos-source-centos-7-nv"},{"line_number":11402,"context_line":"      - gate-kolla-ansible-dsvm-deploy-ubuntu-binary-ubuntu-xenial-nv"},{"line_number":11403,"context_line":"      - gate-kolla-ansible-dsvm-deploy-ubuntu-source-ubuntu-xenial-nv"},{"line_number":11404,"context_line":"      - gate-kolla-ansible-dsvm-deploy-oraclelinux-binary-centos-7-nv"},{"line_number":11405,"context_line":"      - gate-kolla-ansible-dsvm-deploy-oraclelinux-source-centos-7-nv"},{"line_number":11406,"context_line":"      - gate-kolla-ansible-dsvm-deploy-multinode-ubuntu-source-ubuntu-xenial-2-node-nv"},{"line_number":11407,"context_line":"      - gate-kolla-ansible-dsvm-deploy-multinode-ubuntu-binary-ubuntu-xenial-2-node-nv"},{"line_number":11408,"context_line":"      - gate-kolla-ansible-dsvm-deploy-multinode-centos-source-centos-7-2-node-nv"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"7f231b9d_70682db8","line":11405,"range":{"start_line":11400,"start_character":0,"end_line":11405,"end_character":69},"updated":"2017-06-14 17:32:13.000000000","message":"It looks like these jobs are redundant now and should be removed?","commit_id":"95d2c02e84b8da5ff4940745e817423cdd212fb3"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"4308190ca9b42ddb3fe6b2afcc4e2e09c829ae21","unresolved":false,"context_lines":[{"line_number":11397,"context_line":"      - name: release-notes-jobs"},{"line_number":11398,"context_line":"      - name: deploy-guide-jobs"},{"line_number":11399,"context_line":"    check:"},{"line_number":11400,"context_line":"      - gate-kolla-ansible-dsvm-deploy-centos-binary-centos-7-nv"},{"line_number":11401,"context_line":"      - gate-kolla-ansible-dsvm-deploy-centos-source-centos-7-nv"},{"line_number":11402,"context_line":"      - gate-kolla-ansible-dsvm-deploy-ubuntu-binary-ubuntu-xenial-nv"},{"line_number":11403,"context_line":"      - gate-kolla-ansible-dsvm-deploy-ubuntu-source-ubuntu-xenial-nv"},{"line_number":11404,"context_line":"      - gate-kolla-ansible-dsvm-deploy-oraclelinux-binary-centos-7-nv"},{"line_number":11405,"context_line":"      - gate-kolla-ansible-dsvm-deploy-oraclelinux-source-centos-7-nv"},{"line_number":11406,"context_line":"      - gate-kolla-ansible-dsvm-deploy-multinode-ubuntu-source-ubuntu-xenial-2-node-nv"},{"line_number":11407,"context_line":"      - gate-kolla-ansible-dsvm-deploy-multinode-ubuntu-binary-ubuntu-xenial-2-node-nv"},{"line_number":11408,"context_line":"      - gate-kolla-ansible-dsvm-deploy-multinode-centos-source-centos-7-2-node-nv"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"7f231b9d_d00ca1ed","line":11405,"range":{"start_line":11400,"start_character":0,"end_line":11405,"end_character":69},"in_reply_to":"7f231b9d_70682db8","updated":"2017-06-14 17:35:30.000000000","message":"they\u0027re aio, other are multinode. Possibly we can remove them but that\u0027s different conversation to have","commit_id":"95d2c02e84b8da5ff4940745e817423cdd212fb3"}]}