)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"7d02d1cfdb40306f3c8eb55695cd0166f5845dd6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3e126b4f_541cdcf0","updated":"2021-11-09 19:08:21.000000000","message":"recheck","commit_id":"8c1e8e472e91adcdcf0fba96376416453dbb0870"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"3a3797159117dec677f3fa4827fd496caebd62b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f40ce745_327ac62b","updated":"2021-11-10 09:14:47.000000000","message":"recheck","commit_id":"8c1e8e472e91adcdcf0fba96376416453dbb0870"}],"zuul.d/project.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"0ef4d30f89eaa6c497c5a813bee5e60f1b3b8008","unresolved":true,"context_lines":[{"line_number":28,"context_line":"      jobs:"},{"line_number":29,"context_line":"        - tripleo-ci-centos-7-standalone-os-tempest"},{"line_number":30,"context_line":"        - openstack-ansible-deploy-aio_metal-centos-7:"},{"line_number":31,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5e4402c6_2de939e3","line":31,"updated":"2021-11-10 00:21:31.000000000","message":"Do not put non voting jobs in the gate. They don\u0027t affect gating and will only serve to consume resources there.","commit_id":"8c1e8e472e91adcdcf0fba96376416453dbb0870"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"f595ab1c531e3d42e188e259d55fdf5a7af78d0f","unresolved":true,"context_lines":[{"line_number":28,"context_line":"      jobs:"},{"line_number":29,"context_line":"        - tripleo-ci-centos-7-standalone-os-tempest"},{"line_number":30,"context_line":"        - openstack-ansible-deploy-aio_metal-centos-7:"},{"line_number":31,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e6f69f98_bd8b7ae7","line":31,"in_reply_to":"5e4402c6_2de939e3","updated":"2021-11-10 09:12:58.000000000","message":"we don\u0027t have much choice here since job is defined as part of the template  openstack-ansible-deploy-aio_metal-jobs and we merge this patch to solve curcular dependency only. Otherwise dependency would be even more awfull.\n\nYou might see https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/817221 that reverts this one and fixes this.","commit_id":"8c1e8e472e91adcdcf0fba96376416453dbb0870"},{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"f6a34f5eafa0a29096db5652c1b2b3a69386b40e","unresolved":true,"context_lines":[{"line_number":28,"context_line":"      jobs:"},{"line_number":29,"context_line":"        - tripleo-ci-centos-7-standalone-os-tempest"},{"line_number":30,"context_line":"        - openstack-ansible-deploy-aio_metal-centos-7:"},{"line_number":31,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"07a344ec_2f0aec9c","line":31,"in_reply_to":"e6f69f98_bd8b7ae7","updated":"2021-11-10 12:02:40.000000000","message":"Yes, this patch is to break a circular dependancy between https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/816473 and https://review.opendev.org/c/openstack/openstack-ansible/+/815633.\n\nThe jobs are defined in templates outside this repo so whilst this looks like the addition of a new NV job it\u0027s actually making an existing job in the template NV for the purpose of deadlock-breaking.\n\nIt\u0027s not really great to make the jobs in the template NV for this as it would open us to merging breaking changes in the several dozen repos which use the template during that time.\n\nIt would be great to be able to do something like this to knock a specific job out of a template in a job which uses it:\n\n    check:\n      skip:\n        - openstack-ansible-deploy-aio_metal-centos-7\n      jobs:\n        - tripleo-ci-centos-7-standalone-os-tempest\n    gate:\n      skip:\n        - openstack-ansible-deploy-aio_metal-centos-7\n      jobs:\n        - tripleo-ci-centos-7-standalone-os-tempest\n\nThis does come up quite a lot for us when we need to deadlock-break, and it\u0027s a result of being multi-repo, using of zuul job templates for good DRY-ness in job definitions. Mono-repos would not have this trouble as you could commit all of the fixes in one hit.","commit_id":"8c1e8e472e91adcdcf0fba96376416453dbb0870"}]}