)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"e92b4e337bdbcaca9d57075065ba95901ccd952f","unresolved":false,"context_lines":[{"line_number":12,"context_line":"The root cause is that when resume a VM with block storage,"},{"line_number":13,"context_line":"libvirt driver needs to call conductor via rpcapi to update"},{"line_number":14,"context_line":"block device, but the function of resume() do not have context,"},{"line_number":15,"context_line":"this will cause RPC api failed."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I712777ed1d893a2b6463d30c407b0a677e37b602"},{"line_number":18,"context_line":"Closes-Bug: #1241337"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAATn%2F%2FEgg%3D","line":15,"updated":"2013-11-14 20:23:40.000000000","message":"the wording is a little ackward here, but ok","commit_id":"20efb9933b24f957cbd6222c036c013e2474c36b"}],"nova/virt/driver.py":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"db04ee59f034593f0dcb86934d7d3a4b525dc86b","unresolved":false,"context_lines":[{"line_number":432,"context_line":"        raise NotImplementedError()"},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"    def resume(self, context, instance, network_info, block_device_info\u003dNone):"},{"line_number":435,"context_line":"        \"\"\"resume the specified instance.\"\"\""},{"line_number":436,"context_line":"        # TODO(Vek): Need to pass context in for access to auth_token"},{"line_number":437,"context_line":"        raise NotImplementedError()"},{"line_number":438,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F1SM%3D","line":435,"updated":"2013-10-28 09:41:05.000000000","message":"Can you document what this \u0027context\u0027 parameter actually is here, and also document the other parameters of this method while you\u0027re here.","commit_id":"c67bfb3c1a60112bf3527cbee451f1d0f710a573"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"12b7d2b2df5875d7a5da2888af6b13a0448427f0","unresolved":false,"context_lines":[{"line_number":432,"context_line":"        raise NotImplementedError()"},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"    def resume(self, context, instance, network_info, block_device_info\u003dNone):"},{"line_number":435,"context_line":"        \"\"\"resume the specified instance.\"\"\""},{"line_number":436,"context_line":"        # TODO(Vek): Need to pass context in for access to auth_token"},{"line_number":437,"context_line":"        raise NotImplementedError()"},{"line_number":438,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F1QA%3D","line":435,"in_reply_to":"AAAATn%2F%2F1SM%3D","updated":"2013-10-28 09:55:22.000000000","message":"Done","commit_id":"c67bfb3c1a60112bf3527cbee451f1d0f710a573"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"bdb59814ee706a69c4304aaed0ec71072da46994","unresolved":false,"context_lines":[{"line_number":3217,"context_line":"        return domain"},{"line_number":3218,"context_line":""},{"line_number":3219,"context_line":"    def _create_domain_and_network(self, xml, instance, network_info,"},{"line_number":3220,"context_line":"                                   block_device_info\u003dNone, power_on\u003dTrue,"},{"line_number":3221,"context_line":"                                   context\u003dNone, reboot\u003dFalse):"},{"line_number":3222,"context_line":""},{"line_number":3223,"context_line":"        \"\"\"Do required network setup and create domain.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Fx2M%3D","line":3220,"updated":"2013-10-29 07:27:53.000000000","message":"I really don\u0027t like that context is an optional argument to this method\n\nIt\u0027s very difficult to verify that a caller which doesn\u0027t pass context isn\u0027t going to hit the same bug as this\n\ne.g. why is it ok for finish_revert_migration() to not pass a context?","commit_id":"6ed02c02ae98942ec175a7238fcd6b2133fed285"}]}
