)]}'
{".zuul.d/centos.yaml":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"change_message_id":"bc29e9df4cf15da17bc23445d6be32f54ced28bd","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- project:"},{"line_number":3,"context_line":"    check:"},{"line_number":4,"context_line":"      jobs:"},{"line_number":5,"context_line":"        - kolla-build-centos-binary"},{"line_number":6,"context_line":"        - kolla-build-centos-source"},{"line_number":7,"context_line":"        - kolla-ansible-centos-source:"},{"line_number":8,"context_line":"            required-projects:"},{"line_number":9,"context_line":"              - openstack/kolla-ansible"},{"line_number":10,"context_line":"              - openstack/requirements"},{"line_number":11,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":12,"context_line":"            required-projects:"},{"line_number":13,"context_line":"              - openstack/kolla-ansible"},{"line_number":14,"context_line":"              - openstack/requirements"},{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"          voting: false"},{"line_number":17,"context_line":"    gate:"},{"line_number":18,"context_line":"      jobs:"},{"line_number":19,"context_line":"        - kolla-build-centos-source"},{"line_number":20,"context_line":"    periodic:"},{"line_number":21,"context_line":"      jobs:"},{"line_number":22,"context_line":"        - kolla-publish-centos-source"},{"line_number":23,"context_line":"        - kolla-publish-centos-binary"},{"line_number":24,"context_line":"    release:"},{"line_number":25,"context_line":"      jobs:"},{"line_number":26,"context_line":"        - kolla-publish-centos-source"},{"line_number":27,"context_line":"        - kolla-publish-centos-binary"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- job:"},{"line_number":30,"context_line":"    name: kolla-build-centos-binary"},{"line_number":31,"context_line":"    parent: kolla-base"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_3351090d","line":28,"range":{"start_line":2,"start_character":2,"end_line":28,"end_character":0},"updated":"2018-11-12 16:54:06.000000000","message":"expected str for dictionary value @ data[\u0027check\u0027][\u0027jobs\u0027][4][\u0027voting\u0027]","commit_id":"f0a30791dcaaf2959232a78ce2662f64e34a7918"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"d89b50dc647fea88f5df9a7d603d3bfb571f2fdd","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^kolla/.*$"},{"line_number":19,"context_line":"    gate:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - kolla-build-centos-source"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f79a3b5_be1876ce","line":18,"updated":"2018-11-13 12:42:33.000000000","message":"Sure you don\u0027t want to trigger on changes to the docker files under docker/?","commit_id":"303117e96206a8bd9c918a70320a7454767c5103"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"d593824ed39408e1e5ed82326c10e93ea8d13d52","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^kolla/.*$"},{"line_number":19,"context_line":"    gate:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - kolla-build-centos-source"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f79a3b5_ca5a4c1e","line":18,"in_reply_to":"3f79a3b5_be1876ce","updated":"2018-11-13 15:07:11.000000000","message":"Indeed. Will update.","commit_id":"303117e96206a8bd9c918a70320a7454767c5103"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"9f772dbf2dda26559e923cb014e0d77aaf0f7637","unresolved":false,"context_lines":[{"line_number":13,"context_line":"              - openstack/kolla-ansible"},{"line_number":14,"context_line":"              - openstack/requirements"},{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^docker/.*$"},{"line_number":19,"context_line":"              - ^kolla/.*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_f1aa3b6b","line":16,"updated":"2018-11-20 13:49:56.000000000","message":"Shouldn\u0027t this add openstack-infra/tripleo-ci as required-projects?","commit_id":"e79da48e176388d5a0e83630ceae6bdada57618c"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"c963e4681d99220b6db5e593010fabfe7f09feb5","unresolved":false,"context_lines":[{"line_number":13,"context_line":"              - openstack/kolla-ansible"},{"line_number":14,"context_line":"              - openstack/requirements"},{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^docker/.*$"},{"line_number":19,"context_line":"              - ^kolla/.*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_871db813","line":16,"in_reply_to":"3f79a3b5_4744801a","updated":"2018-11-21 10:35:26.000000000","message":"Let\u0027s just test it. Can one of you submit a patch based on this one that changes a file in docker/ or kolla/. If the job runs, it works :)","commit_id":"e79da48e176388d5a0e83630ceae6bdada57618c"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"d4290b919dd0357b5c17dc08a3c34c6281a10ae6","unresolved":false,"context_lines":[{"line_number":13,"context_line":"              - openstack/kolla-ansible"},{"line_number":14,"context_line":"              - openstack/requirements"},{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^docker/.*$"},{"line_number":19,"context_line":"              - ^kolla/.*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_59e0dff9","line":16,"in_reply_to":"3f79a3b5_4744801a","updated":"2018-11-21 15:27:26.000000000","message":"zuul is aware of all the job definitions from it\u0027s known repos so it just pulls it in.  It\u0027s one of those zuul magic things :D","commit_id":"e79da48e176388d5a0e83630ceae6bdada57618c"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"a28834ca28987f2aa6b5ed74a0e853f7ee82cddc","unresolved":false,"context_lines":[{"line_number":13,"context_line":"              - openstack/kolla-ansible"},{"line_number":14,"context_line":"              - openstack/requirements"},{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^docker/.*$"},{"line_number":19,"context_line":"              - ^kolla/.*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_a7a554b5","line":16,"in_reply_to":"3f79a3b5_67ddfc3d","updated":"2018-11-21 10:40:37.000000000","message":"Indeed, thanks Mark. Testing with https://review.openstack.org/#/c/619202/","commit_id":"e79da48e176388d5a0e83630ceae6bdada57618c"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"3b2b12d6cf0abec60e77764437e9cf2f218311a8","unresolved":false,"context_lines":[{"line_number":13,"context_line":"              - openstack/kolla-ansible"},{"line_number":14,"context_line":"              - openstack/requirements"},{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^docker/.*$"},{"line_number":19,"context_line":"              - ^kolla/.*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_67ddfc3d","line":16,"in_reply_to":"3f79a3b5_871db813","updated":"2018-11-21 10:36:56.000000000","message":"I think if zuul was not able to find the job it would have complained by now. We have a similar setup in kayobe/kayobe-config-dev, and do not need to specify a required project.","commit_id":"e79da48e176388d5a0e83630ceae6bdada57618c"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"2010e571e9a5a45cc034d72a23c25d3f5993d019","unresolved":false,"context_lines":[{"line_number":13,"context_line":"              - openstack/kolla-ansible"},{"line_number":14,"context_line":"              - openstack/requirements"},{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^docker/.*$"},{"line_number":19,"context_line":"              - ^kolla/.*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_fbb3f597","line":16,"in_reply_to":"3f79a3b5_f1aa3b6b","updated":"2018-11-20 17:27:09.000000000","message":"It should be inherited because it\u0027s in the job definition. I don\u0027t think we need it explicitly.\n\nhttps://review.openstack.org/#/c/614392/31/zuul.d/build-containers.yaml","commit_id":"e79da48e176388d5a0e83630ceae6bdada57618c"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"ce5d7ba65dc84f01682c3702c3f3a6e9d541534c","unresolved":false,"context_lines":[{"line_number":13,"context_line":"              - openstack/kolla-ansible"},{"line_number":14,"context_line":"              - openstack/requirements"},{"line_number":15,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"            files:"},{"line_number":18,"context_line":"              - ^docker/.*$"},{"line_number":19,"context_line":"              - ^kolla/.*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_4744801a","line":16,"in_reply_to":"3f79a3b5_fbb3f597","updated":"2018-11-21 10:31:33.000000000","message":"But how does kolla get the job definition from openstack-infra/tripleo-ci? IIUC we need to point to the repo where the tripleo-build-containers-centos-7 job lives since it\u0027s neither in openstack-infra/zuul-jobs or openstack-infra/openstack-zuul-jobs.","commit_id":"e79da48e176388d5a0e83630ceae6bdada57618c"}],"/COMMIT_MSG":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"9f772dbf2dda26559e923cb014e0d77aaf0f7637","unresolved":false,"context_lines":[{"line_number":11,"context_line":"integration between the two.  This job runs the tripleo build process"},{"line_number":12,"context_line":"which calls kolla-build under the covers.  Failures in this job might"},{"line_number":13,"context_line":"indicate breakages in the kolla build api."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5ce7c5888858f535eee0b5539c1e2d7978a7438e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3f79a3b5_11fc976f","line":14,"updated":"2018-11-20 13:49:56.000000000","message":"Should we perhaps reference the original patch I97ceb9533a38ec7b175623784649de277aaf4274 that added tripleo-build-containers-centos-7 job to tripleo-ci ?","commit_id":"e79da48e176388d5a0e83630ceae6bdada57618c"}]}