)]}'
{".zuul.d/centos.yaml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"62aa1b66c86452673a9a6bdf81e71902116c4af3","unresolved":false,"context_lines":[{"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":"            voting: false"},{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_27207bf7","line":8,"range":{"start_line":8,"start_character":12,"end_line":8,"end_character":25},"updated":"2019-08-14 09:23:58.000000000","message":"This doesn\u0027t make sense. We shouldn\u0027t allow patches through to the gate if they have failed jobs required to pass the gate.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"438d1c88fb689141a4ac95552f4f10afdf7b221f","unresolved":false,"context_lines":[{"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":"            voting: false"},{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_87ab6f1c","line":8,"range":{"start_line":8,"start_character":12,"end_line":8,"end_character":25},"in_reply_to":"7faddb67_27207bf7","updated":"2019-08-14 09:34:16.000000000","message":"But we can ensure they eventually pass the gate when we have a related change in k-a.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"29b78c70adfa2f0c91cf9275432af9eb0989026c","unresolved":false,"context_lines":[{"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":"            voting: false"},{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_9655dffc","line":8,"range":{"start_line":8,"start_character":12,"end_line":8,"end_character":25},"in_reply_to":"7faddb67_87ab6f1c","updated":"2019-08-14 13:23:52.000000000","message":"Sure, but failing the gate is expensive since it may include a number of patches. It\u0027s bad etiquette to allow a patch through to the gate that may fail.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"62aa1b66c86452673a9a6bdf81e71902116c4af3","unresolved":false,"context_lines":[{"line_number":6,"context_line":"        - kolla-build-centos-source"},{"line_number":7,"context_line":"        - kolla-ansible-centos-source:"},{"line_number":8,"context_line":"            voting: false"},{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"},{"line_number":12,"context_line":"            voting: false"},{"line_number":13,"context_line":"        - tripleo-build-containers-centos-7:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_e7c5830b","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":25},"updated":"2019-08-14 09:23:58.000000000","message":"This is defined as non-voting jobs in kolla-ansible - not necessary.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"29b78c70adfa2f0c91cf9275432af9eb0989026c","unresolved":false,"context_lines":[{"line_number":6,"context_line":"        - kolla-build-centos-source"},{"line_number":7,"context_line":"        - kolla-ansible-centos-source:"},{"line_number":8,"context_line":"            voting: false"},{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"},{"line_number":12,"context_line":"            voting: false"},{"line_number":13,"context_line":"        - tripleo-build-containers-centos-7:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_766383de","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":25},"in_reply_to":"7faddb67_678753b8","updated":"2019-08-14 13:23:52.000000000","message":"I don\u0027t follow.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"8283b561a653127fdce723c74c1900076fdbab4d","unresolved":false,"context_lines":[{"line_number":6,"context_line":"        - kolla-build-centos-source"},{"line_number":7,"context_line":"        - kolla-ansible-centos-source:"},{"line_number":8,"context_line":"            voting: false"},{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"},{"line_number":12,"context_line":"            voting: false"},{"line_number":13,"context_line":"        - tripleo-build-containers-centos-7:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_8c0cee94","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":25},"in_reply_to":"7faddb67_766383de","updated":"2019-08-14 14:58:31.000000000","message":"Never mind, I made up my mind that I don\u0027t want this solution. I meant apply voting: false to any job from k-a so that possible new jobs are introduced the same even if k-a changes them to voting.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"438d1c88fb689141a4ac95552f4f10afdf7b221f","unresolved":false,"context_lines":[{"line_number":6,"context_line":"        - kolla-build-centos-source"},{"line_number":7,"context_line":"        - kolla-ansible-centos-source:"},{"line_number":8,"context_line":"            voting: false"},{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"},{"line_number":12,"context_line":"            voting: false"},{"line_number":13,"context_line":"        - tripleo-build-containers-centos-7:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_678753b8","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":25},"in_reply_to":"7faddb67_e7c5830b","updated":"2019-08-14 09:34:16.000000000","message":"This is to remember to follow the schema.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"62aa1b66c86452673a9a6bdf81e71902116c4af3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"},{"line_number":12,"context_line":"            voting: false"},{"line_number":13,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":14,"context_line":"            voting: true"},{"line_number":15,"context_line":"            files:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_c7d2c7d1","line":12,"range":{"start_line":12,"start_character":12,"end_line":12,"end_character":25},"updated":"2019-08-14 09:23:58.000000000","message":"As first job.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"438d1c88fb689141a4ac95552f4f10afdf7b221f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"        - kolla-ansible-centos-binary:"},{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - kolla-ansible-centos-source-upgrade:"},{"line_number":12,"context_line":"            voting: false"},{"line_number":13,"context_line":"        - tripleo-build-containers-centos-7:"},{"line_number":14,"context_line":"            voting: true"},{"line_number":15,"context_line":"            files:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_87828fa4","line":12,"range":{"start_line":12,"start_character":12,"end_line":12,"end_character":25},"in_reply_to":"7faddb67_c7d2c7d1","updated":"2019-08-14 09:34:16.000000000","message":"ditto","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"62aa1b66c86452673a9a6bdf81e71902116c4af3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"              - ^setup.cfg$"},{"line_number":21,"context_line":"              - ^setup.py$"},{"line_number":22,"context_line":"        - kolla-ansible-bifrost-centos-source:"},{"line_number":23,"context_line":"            files: ^docker\\/(base|bifrost|openstack-base)\\/.*"},{"line_number":24,"context_line":"            voting: false"},{"line_number":25,"context_line":"        # Test rabbitmq \u0026 mariadb changes in multinode ceph jobs."},{"line_number":26,"context_line":"        - kolla-ansible-centos-source-ceph:"},{"line_number":27,"context_line":"            files: ^docker\\/(base|ceph|cinder|glance|mariadb|openstack-base|rabbitmq)\\/.*"},{"line_number":28,"context_line":"            voting: false"},{"line_number":29,"context_line":"        - kolla-ansible-centos-source-upgrade-ceph:"},{"line_number":30,"context_line":"            files: ^docker\\/(base|ceph|cinder|glance|mariadb|openstack-base|rabbitmq)\\/.*"},{"line_number":31,"context_line":"            voting: false"},{"line_number":32,"context_line":"        - kolla-ansible-centos-source-cinder-lvm:"},{"line_number":33,"context_line":"            files: ^docker\\/(base|cinder|iscsid|openstack-base)\\/.*"},{"line_number":34,"context_line":"            voting: false"},{"line_number":35,"context_line":"        - kolla-ansible-centos-source-scenario-nfv:"},{"line_number":36,"context_line":"            files: ^docker\\/(base|barbican|heat|mistral|openstack-base|redis|tacker)\\/.*"},{"line_number":37,"context_line":"            voting: false"},{"line_number":38,"context_line":"        - kolla-ansible-centos-source-ironic:"},{"line_number":39,"context_line":"            files: ^docker\\/(base|dnsmasq|ironic|ironic-inspector|iscsid|openstack-base)\\/.*"},{"line_number":40,"context_line":"            voting: false"},{"line_number":41,"context_line":"    gate:"},{"line_number":42,"context_line":"      queue: kolla"},{"line_number":43,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_87660f55","line":40,"range":{"start_line":23,"start_character":0,"end_line":40,"end_character":25},"updated":"2019-08-14 09:23:58.000000000","message":"These are all defined as non-voting jobs in kolla-ansible - not necessary.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"438d1c88fb689141a4ac95552f4f10afdf7b221f","unresolved":false,"context_lines":[{"line_number":20,"context_line":"              - ^setup.cfg$"},{"line_number":21,"context_line":"              - ^setup.py$"},{"line_number":22,"context_line":"        - kolla-ansible-bifrost-centos-source:"},{"line_number":23,"context_line":"            files: ^docker\\/(base|bifrost|openstack-base)\\/.*"},{"line_number":24,"context_line":"            voting: false"},{"line_number":25,"context_line":"        # Test rabbitmq \u0026 mariadb changes in multinode ceph jobs."},{"line_number":26,"context_line":"        - kolla-ansible-centos-source-ceph:"},{"line_number":27,"context_line":"            files: ^docker\\/(base|ceph|cinder|glance|mariadb|openstack-base|rabbitmq)\\/.*"},{"line_number":28,"context_line":"            voting: false"},{"line_number":29,"context_line":"        - kolla-ansible-centos-source-upgrade-ceph:"},{"line_number":30,"context_line":"            files: ^docker\\/(base|ceph|cinder|glance|mariadb|openstack-base|rabbitmq)\\/.*"},{"line_number":31,"context_line":"            voting: false"},{"line_number":32,"context_line":"        - kolla-ansible-centos-source-cinder-lvm:"},{"line_number":33,"context_line":"            files: ^docker\\/(base|cinder|iscsid|openstack-base)\\/.*"},{"line_number":34,"context_line":"            voting: false"},{"line_number":35,"context_line":"        - kolla-ansible-centos-source-scenario-nfv:"},{"line_number":36,"context_line":"            files: ^docker\\/(base|barbican|heat|mistral|openstack-base|redis|tacker)\\/.*"},{"line_number":37,"context_line":"            voting: false"},{"line_number":38,"context_line":"        - kolla-ansible-centos-source-ironic:"},{"line_number":39,"context_line":"            files: ^docker\\/(base|dnsmasq|ironic|ironic-inspector|iscsid|openstack-base)\\/.*"},{"line_number":40,"context_line":"            voting: false"},{"line_number":41,"context_line":"    gate:"},{"line_number":42,"context_line":"      queue: kolla"},{"line_number":43,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_478c97d6","line":40,"range":{"start_line":23,"start_character":0,"end_line":40,"end_character":25},"in_reply_to":"7faddb67_87660f55","updated":"2019-08-14 09:34:16.000000000","message":"ditto","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"}],"/COMMIT_MSG":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"edfa95ba97204ca1fe0652c39b3a37b7b2f64ebe","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We should gate on kolla-ansible jobs. Period."},{"line_number":10,"context_line":"This makes kolla gate on jobs that kolla-ansible gates on already."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Making k-a jobs non-voting in checks should let us merge changes that"},{"line_number":13,"context_line":"need a patch in kolla-ansible as well but otherwise cyclic deps"},{"line_number":14,"context_line":"are not allowed in CI."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Iee51a5c340d071e653afd4ecd4996d04bd9cfb7f"},{"line_number":17,"context_line":"Signed-off-by: Radosław Piliszek \u003cradoslaw.piliszek@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7faddb67_e7eee389","line":14,"range":{"start_line":12,"start_character":0,"end_line":14,"end_character":22},"updated":"2019-08-14 09:25:22.000000000","message":"IMO cyclic dependency just shows we have a breaking change, which will affect users.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"438d1c88fb689141a4ac95552f4f10afdf7b221f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We should gate on kolla-ansible jobs. Period."},{"line_number":10,"context_line":"This makes kolla gate on jobs that kolla-ansible gates on already."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Making k-a jobs non-voting in checks should let us merge changes that"},{"line_number":13,"context_line":"need a patch in kolla-ansible as well but otherwise cyclic deps"},{"line_number":14,"context_line":"are not allowed in CI."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Iee51a5c340d071e653afd4ecd4996d04bd9cfb7f"},{"line_number":17,"context_line":"Signed-off-by: Radosław Piliszek \u003cradoslaw.piliszek@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7faddb67_0733ff14","line":14,"range":{"start_line":12,"start_character":0,"end_line":14,"end_character":22},"in_reply_to":"7faddb67_e7eee389","updated":"2019-08-14 09:34:16.000000000","message":"We do. Any ideas how to approach that? Pending issues include HAProxy update for CentOS and td-agent update, also for CentOS. Slap temporary workarounds? I am unsure whether we should do what I propose (not really sure Zuul knows what to do in that case, it could as well just ignore the other changes in gate) but have no better idea atm.","commit_id":"8f28396b8932d9a0a56971416163ebac6361e344"}]}