)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"82a9498362ceb386270eaa6a044f196c60be85c3","unresolved":false,"context_lines":[{"line_number":12,"context_line":"Remove the \u0027branches:\u0027 specification on all jobs except the gate job"},{"line_number":13,"context_line":"\u0027ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa\u0027 as"},{"line_number":14,"context_line":"there appears to be an issue during the migration for that job. More"},{"line_number":15,"context_line":"research is needed on what to do for that job."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Also remove the jobs from zuul.d/legacy-ironic-jobs.yaml and"},{"line_number":18,"context_line":"corresponding files in playbooks/legacy/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3f4b6375_e0d40622","line":15,"updated":"2017-11-06 23:27:59.000000000","message":"Except for the first part, the rest is wrong since the patches were re-ordered.","commit_id":"b0d06f8109175237ba5c1ec7ea36572d5e4b0f8d"}],"zuul.d/project.yaml":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"fc73a15bc1e6275409e8d7f2b7efc1f8d54225ca","unresolved":false,"context_lines":[{"line_number":22,"context_line":"      queue: ironic"},{"line_number":23,"context_line":"      jobs:"},{"line_number":24,"context_line":"        - ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa"},{"line_number":25,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":26,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":27,"context_line":"        - ironic-tempest-dsvm-ipa-partition-redfish-tinyipa"},{"line_number":28,"context_line":"        - ironic-tempest-dsvm-bfv"},{"line_number":29,"context_line":"        - ironic-dsvm-standalone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_dfd2315c","line":26,"range":{"start_line":25,"start_character":9,"end_line":26,"end_character":47},"updated":"2017-11-03 17:55:36.000000000","message":"I think we need to remove the branches here.","commit_id":"662ff24cae7ffbcd9fd376c24f838d32cf9a5f44"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"734cfaa863ea3034cd65b5b7c28d8db6c4bb4d0b","unresolved":false,"context_lines":[{"line_number":23,"context_line":"      jobs:"},{"line_number":24,"context_line":"        - ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa"},{"line_number":25,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":26,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":27,"context_line":"        - ironic-tempest-dsvm-ipa-partition-redfish-tinyipa"},{"line_number":28,"context_line":"        - ironic-tempest-dsvm-bfv"},{"line_number":29,"context_line":"        - ironic-dsvm-standalone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_c7bee763","line":26,"in_reply_to":"3f4b6375_07519fd9","updated":"2017-11-03 20:12:25.000000000","message":"I agree with you. I would just rather do it in a separate patch. Which I can do.","commit_id":"662ff24cae7ffbcd9fd376c24f838d32cf9a5f44"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"a70b628180adac2566f11ab15cb7b44fd49fd91c","unresolved":false,"context_lines":[{"line_number":23,"context_line":"      jobs:"},{"line_number":24,"context_line":"        - ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa"},{"line_number":25,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":26,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":27,"context_line":"        - ironic-tempest-dsvm-ipa-partition-redfish-tinyipa"},{"line_number":28,"context_line":"        - ironic-tempest-dsvm-bfv"},{"line_number":29,"context_line":"        - ironic-dsvm-standalone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_07519fd9","line":26,"in_reply_to":"3f4b6375_14c66b69","updated":"2017-11-03 19:43:37.000000000","message":"i get that, this is what I think needs to be done here.","commit_id":"662ff24cae7ffbcd9fd376c24f838d32cf9a5f44"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"ae4c8220ccd2ee2067bd72d9dbaec379ee92bc31","unresolved":false,"context_lines":[{"line_number":23,"context_line":"      jobs:"},{"line_number":24,"context_line":"        - ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa"},{"line_number":25,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":26,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":27,"context_line":"        - ironic-tempest-dsvm-ipa-partition-redfish-tinyipa"},{"line_number":28,"context_line":"        - ironic-tempest-dsvm-bfv"},{"line_number":29,"context_line":"        - ironic-dsvm-standalone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_275dbb46","line":26,"in_reply_to":"3f4b6375_c7bee763","updated":"2017-11-03 20:16:20.000000000","message":"Follow-on patch proposed to do that:\n\nhttps://review.openstack.org/#/c/517719/","commit_id":"662ff24cae7ffbcd9fd376c24f838d32cf9a5f44"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"78fd4b00320a6c8a9d1960d63539cdcd588c646a","unresolved":false,"context_lines":[{"line_number":22,"context_line":"      queue: ironic"},{"line_number":23,"context_line":"      jobs:"},{"line_number":24,"context_line":"        - ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa"},{"line_number":25,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":26,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":27,"context_line":"        - ironic-tempest-dsvm-ipa-partition-redfish-tinyipa"},{"line_number":28,"context_line":"        - ironic-tempest-dsvm-bfv"},{"line_number":29,"context_line":"        - ironic-dsvm-standalone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_14c66b69","line":26,"range":{"start_line":25,"start_character":9,"end_line":26,"end_character":47},"in_reply_to":"3f4b6375_dfd2315c","updated":"2017-11-03 19:04:48.000000000","message":"That is explained in the commit message:\n\n Remove the \u0027branches:\u0027 specification on all jobs except the gate job\n \u0027ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa\u0027 as\n there appears to be an issue during the migration for that job. More\n research is needed on what to do for that job.","commit_id":"662ff24cae7ffbcd9fd376c24f838d32cf9a5f44"}]}