)]}'
{"ironic_python_agent/tests/unit/extensions/test_standby.py":[{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"4a5feb421b627bb4fe22a18f55ad6e66e2d40b73","unresolved":false,"context_lines":[{"line_number":375,"context_line":"                          standby._verify_image,"},{"line_number":376,"context_line":"                          image_info, image_location, checksum)"},{"line_number":377,"context_line":""},{"line_number":378,"context_line":"    @mock.patch(\u0027ironic_lib.disk_utils.get_disk_identifier\u0027, autospec\u003dTrue)"},{"line_number":379,"context_line":"    @mock.patch(\u0027ironic_python_agent.hardware.dispatch_to_managers\u0027,"},{"line_number":380,"context_line":"                autospec\u003dTrue)"},{"line_number":381,"context_line":"    @mock.patch(\u0027ironic_python_agent.extensions.standby._write_image\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_1e68c8b6","line":378,"updated":"2017-09-28 20:26:42.000000000","message":"nit: could do this with a lambda\n\n @mock.patch(\u0027ironic_lib_disk_utils.get_disk_identifier\u0027, lambda dev: \u0027ROOT\u0027)\n\nThen don\u0027t need L387 or L392","commit_id":"eeb029a627b491b40209021bfe2f6b0a428b48d7"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"24084d37d6403401363da933dce7f2f015a83332","unresolved":false,"context_lines":[{"line_number":375,"context_line":"                          standby._verify_image,"},{"line_number":376,"context_line":"                          image_info, image_location, checksum)"},{"line_number":377,"context_line":""},{"line_number":378,"context_line":"    @mock.patch(\u0027ironic_lib.disk_utils.get_disk_identifier\u0027, autospec\u003dTrue)"},{"line_number":379,"context_line":"    @mock.patch(\u0027ironic_python_agent.hardware.dispatch_to_managers\u0027,"},{"line_number":380,"context_line":"                autospec\u003dTrue)"},{"line_number":381,"context_line":"    @mock.patch(\u0027ironic_python_agent.extensions.standby._write_image\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_d4c9e158","line":378,"in_reply_to":"7f515b1d_1e68c8b6","updated":"2017-09-29 14:48:28.000000000","message":"Done","commit_id":"eeb029a627b491b40209021bfe2f6b0a428b48d7"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"4a5feb421b627bb4fe22a18f55ad6e66e2d40b73","unresolved":false,"context_lines":[{"line_number":400,"context_line":"        self.assertEqual(\u0027SUCCEEDED\u0027, async_result.command_status)"},{"line_number":401,"context_line":"        self.assertIn(\u0027result\u0027, async_result.command_result)"},{"line_number":402,"context_line":"        cmd_result \u003d (\u0027cache_image: image ({}) cached to device {} \u0027"},{"line_number":403,"context_line":"                      \u0027root_uuid\u003dROOT\u0027).format(image_info[\u0027id\u0027], \u0027manager\u0027)"},{"line_number":404,"context_line":"        self.assertEqual(cmd_result, async_result.command_result[\u0027result\u0027])"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":"    @mock.patch(\u0027ironic_python_agent.hardware.dispatch_to_managers\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_feb5ec32","line":403,"updated":"2017-09-28 20:26:42.000000000","message":"nit: seems like this should be:\n\n \u0027root_uuid\u003d{}\u0027).format(...., \u0027ROOT\u0027)\n\nTo be consistent with how it is currently done.","commit_id":"eeb029a627b491b40209021bfe2f6b0a428b48d7"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"24084d37d6403401363da933dce7f2f015a83332","unresolved":false,"context_lines":[{"line_number":400,"context_line":"        self.assertEqual(\u0027SUCCEEDED\u0027, async_result.command_status)"},{"line_number":401,"context_line":"        self.assertIn(\u0027result\u0027, async_result.command_result)"},{"line_number":402,"context_line":"        cmd_result \u003d (\u0027cache_image: image ({}) cached to device {} \u0027"},{"line_number":403,"context_line":"                      \u0027root_uuid\u003dROOT\u0027).format(image_info[\u0027id\u0027], \u0027manager\u0027)"},{"line_number":404,"context_line":"        self.assertEqual(cmd_result, async_result.command_result[\u0027result\u0027])"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":"    @mock.patch(\u0027ironic_python_agent.hardware.dispatch_to_managers\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_14e9b9b5","line":403,"in_reply_to":"7f515b1d_feb5ec32","updated":"2017-09-29 14:48:28.000000000","message":"Done","commit_id":"eeb029a627b491b40209021bfe2f6b0a428b48d7"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"c89c8ee328d4c4258f5a01064cc1cfc97e4e0784","unresolved":false,"context_lines":[{"line_number":429,"context_line":"                                             \u0027root_uuid\u0027)"},{"line_number":430,"context_line":"        self.assertEqual(cmd_result, async_result.command_result[\u0027result\u0027])"},{"line_number":431,"context_line":""},{"line_number":432,"context_line":"    @mock.patch(\u0027ironic_lib.disk_utils.get_disk_identifier\u0027, autospec\u003dTrue)"},{"line_number":433,"context_line":"    @mock.patch(\u0027ironic_python_agent.hardware.dispatch_to_managers\u0027,"},{"line_number":434,"context_line":"                autospec\u003dTrue)"},{"line_number":435,"context_line":"    @mock.patch(\u0027ironic_python_agent.extensions.standby._write_image\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_cba7b74f","line":432,"updated":"2017-10-18 20:10:04.000000000","message":"Can this one and all the others be done like line 378 and use the lambda?","commit_id":"5440bb54c4c5c5c6afc0a1b30ce8511fcb03849e"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"4d47da608420a24b5d3e33de326f934e1075bf11","unresolved":false,"context_lines":[{"line_number":429,"context_line":"                                             \u0027root_uuid\u0027)"},{"line_number":430,"context_line":"        self.assertEqual(cmd_result, async_result.command_result[\u0027result\u0027])"},{"line_number":431,"context_line":""},{"line_number":432,"context_line":"    @mock.patch(\u0027ironic_lib.disk_utils.get_disk_identifier\u0027, autospec\u003dTrue)"},{"line_number":433,"context_line":"    @mock.patch(\u0027ironic_python_agent.hardware.dispatch_to_managers\u0027,"},{"line_number":434,"context_line":"                autospec\u003dTrue)"},{"line_number":435,"context_line":"    @mock.patch(\u0027ironic_python_agent.extensions.standby._write_image\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_e511095f","line":432,"in_reply_to":"3f4b6375_cba7b74f","updated":"2017-10-24 08:09:19.000000000","message":"Done","commit_id":"5440bb54c4c5c5c6afc0a1b30ce8511fcb03849e"}],"releasenotes/notes/fix-to-pass-root-uuid-for-whole-disk-image-1c13b70f6b74bce0.yaml":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"83d55602022aacf6df3c85e1558ccf846bb1d6da","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fixes an issue wherein IPA does not return ``root_uuid`` as part of"},{"line_number":5,"context_line":"    command status when provisioning of whole disk image is done using"},{"line_number":6,"context_line":"    ``agent`` deploy interface from ironic. See `bug 1713916"},{"line_number":7,"context_line":"    \u003chttps://bugs.launchpad.net/ironic/+bug/1713916\u003e`_ for details."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f515b1d_42266817","line":5,"range":{"start_line":5,"start_character":57,"end_line":5,"end_character":64},"updated":"2017-09-28 15:07:47.000000000","message":"nit: s/is done/is performed/","commit_id":"eeb029a627b491b40209021bfe2f6b0a428b48d7"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"24084d37d6403401363da933dce7f2f015a83332","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fixes an issue wherein IPA does not return ``root_uuid`` as part of"},{"line_number":5,"context_line":"    command status when provisioning of whole disk image is done using"},{"line_number":6,"context_line":"    ``agent`` deploy interface from ironic. See `bug 1713916"},{"line_number":7,"context_line":"    \u003chttps://bugs.launchpad.net/ironic/+bug/1713916\u003e`_ for details."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f515b1d_540cd1a3","line":5,"range":{"start_line":5,"start_character":57,"end_line":5,"end_character":64},"in_reply_to":"7f515b1d_42266817","updated":"2017-09-29 14:48:28.000000000","message":"Done","commit_id":"eeb029a627b491b40209021bfe2f6b0a428b48d7"}]}