)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"1b256c784ad29b44c5f9590d57b459ba19ecc59f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Support for abort an ongoing live migration"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In Nova API microversion 2.24 there is a new operation which allows to"},{"line_number":10,"context_line":"abort a running live migration: https://review.openstack.org/277971"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change is to enable this feature at the client side implementing a"},{"line_number":13,"context_line":"new method to call the new nova API:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bae84128_5a711577","line":10,"range":{"start_line":10,"start_character":32,"end_line":10,"end_character":67},"updated":"2016-02-24 19:08:54.000000000","message":"Please, transform reference to Nova\u0027s change to \"Depends-On: \u003cchange-id of Nova\u0027s change\u003e\"","commit_id":"29ff1c47187e3ddbb4b80ef76e9de8e4bdb148b8"},{"author":{"_account_id":12299,"name":"Paweł Koniszewski","email":"pawel.koniszewski@gmail.com","username":"pawel.koniszewski"},"change_message_id":"ec7151baf1c550263ea8a5f8f3c68418366312ed","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Support for abort an ongoing live migration"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In Nova API microversion 2.24 there is a new operation which allows to"},{"line_number":10,"context_line":"abort a running live migration."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change is to enable this feature at the client side implementing a"},{"line_number":13,"context_line":"new method to call the new nova API:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bae84128_958e9d25","line":10,"updated":"2016-02-25 13:51:55.000000000","message":"In case you re-spin please add link to the change in nova","commit_id":"2aa6cfccb9329275a7fc8b80e4a4acddbc2b270b"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"c43748cca419205e1655c4d769504386b755af56","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Support for abort an ongoing live migration"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In Nova API microversion 2.24 there is a new operation which allows to"},{"line_number":10,"context_line":"abort a running live migration: https://review.openstack.org/277971"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change is to enable this feature at the client side implementing a"},{"line_number":13,"context_line":"new method to call the new nova API:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bae84128_4758e0fa","line":10,"updated":"2016-02-25 16:48:55.000000000","message":"*shrug* a little redundant to have both this and the Depends-On, but no worries.","commit_id":"061e22739c70a19da928429fea0aa5b2c660b1a0"},{"author":{"_account_id":1865,"name":"Andrea Rosa","email":"andrea.rosa@gmail.com","username":"andrea-rosa-m"},"change_message_id":"3c2daa7f317e33952b530b0eaeb62f45854e55ef","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Support for abort an ongoing live migration"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In Nova API microversion 2.24 there is a new operation which allows to"},{"line_number":10,"context_line":"abort a running live migration: https://review.openstack.org/277971"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change is to enable this feature at the client side implementing a"},{"line_number":13,"context_line":"new method to call the new nova API:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9aed3d3a_a99ed7b5","line":10,"in_reply_to":"bae84128_4758e0fa","updated":"2016-02-26 18:20:32.000000000","message":"Done","commit_id":"061e22739c70a19da928429fea0aa5b2c660b1a0"}],"novaclient/tests/unit/v2/test_server_migrations.py":[{"author":{"_account_id":12299,"name":"Paweł Koniszewski","email":"pawel.koniszewski@gmail.com","username":"pawel.koniszewski"},"change_message_id":"ec7151baf1c550263ea8a5f8f3c68418366312ed","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        self.assert_called(\u0027GET\u0027, \u0027/servers/1234/migrations/1\u0027)"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"class ServerMigrationsTestV224(ServerMigrationsTest):"},{"line_number":87,"context_line":"    def setUp(self):"},{"line_number":88,"context_line":"        super(ServerMigrationsTestV224, self).setUp()"},{"line_number":89,"context_line":"        self.cs.api_version \u003d api_versions.APIVersion(\"2.24\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"bae84128_55db751b","line":86,"range":{"start_line":86,"start_character":31,"end_line":86,"end_character":51},"updated":"2016-02-25 13:51:55.000000000","message":"This should inherit from ServerMigrationsTestV223 I think","commit_id":"2aa6cfccb9329275a7fc8b80e4a4acddbc2b270b"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"8eb96a47ff7b8aab2b60b7d0eddc17625d76b8ab","unresolved":false,"context_lines":[{"line_number":87,"context_line":"        self.cs.api_version \u003d api_versions.APIVersion(\"2.24\")"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_live_migration_abort(self):"},{"line_number":90,"context_line":"        self.cs.api_version \u003d api_versions.APIVersion(\"2.24\")"},{"line_number":91,"context_line":"        self.cs.server_migrations.live_migration_abort(1234, 1)"},{"line_number":92,"context_line":"        self.assert_called(\u0027DELETE\u0027, \u0027/servers/1234/migrations/1\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9aed3d3a_b94b6686","line":90,"range":{"start_line":90,"start_character":8,"end_line":90,"end_character":61},"updated":"2016-02-29 18:34:51.000000000","message":"it is redundant(see L87)","commit_id":"3f08c23c47498765d45ca26f9a0a6e3cce1ce8b5"},{"author":{"_account_id":7461,"name":"Paul Murray","email":"pmurray@hpe.com","username":"ptm"},"change_message_id":"437354ccb4af8b4bf9e066760bf6fbbe5c635a1a","unresolved":false,"context_lines":[{"line_number":87,"context_line":"        self.cs.api_version \u003d api_versions.APIVersion(\"2.24\")"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_live_migration_abort(self):"},{"line_number":90,"context_line":"        self.cs.api_version \u003d api_versions.APIVersion(\"2.24\")"},{"line_number":91,"context_line":"        self.cs.server_migrations.live_migration_abort(1234, 1)"},{"line_number":92,"context_line":"        self.assert_called(\u0027DELETE\u0027, \u0027/servers/1234/migrations/1\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9aed3d3a_5c7e676a","line":90,"range":{"start_line":90,"start_character":8,"end_line":90,"end_character":61},"in_reply_to":"9aed3d3a_5b503b32","updated":"2016-03-01 08:37:33.000000000","message":"This is trivial and this patch had +W - a follow on would have been better at this stage","commit_id":"3f08c23c47498765d45ca26f9a0a6e3cce1ce8b5"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"3debfdb9ebb24b438fa033698d2c5c3e583d17ad","unresolved":false,"context_lines":[{"line_number":87,"context_line":"        self.cs.api_version \u003d api_versions.APIVersion(\"2.24\")"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_live_migration_abort(self):"},{"line_number":90,"context_line":"        self.cs.api_version \u003d api_versions.APIVersion(\"2.24\")"},{"line_number":91,"context_line":"        self.cs.server_migrations.live_migration_abort(1234, 1)"},{"line_number":92,"context_line":"        self.assert_called(\u0027DELETE\u0027, \u0027/servers/1234/migrations/1\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9aed3d3a_5b503b32","line":90,"range":{"start_line":90,"start_character":8,"end_line":90,"end_character":61},"in_reply_to":"9aed3d3a_b94b6686","updated":"2016-03-01 02:30:07.000000000","message":"Done","commit_id":"3f08c23c47498765d45ca26f9a0a6e3cce1ce8b5"}],"novaclient/v2/server_migrations.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"8eb96a47ff7b8aab2b60b7d0eddc17625d76b8ab","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        :param migration: Migration id that will be cancelled"},{"line_number":77,"context_line":"        :returns: An instance of novaclient.base.TupleWithMeta"},{"line_number":78,"context_line":"        \"\"\""},{"line_number":79,"context_line":"        resp, body \u003d self.api.client.delete("},{"line_number":80,"context_line":"            \u0027/servers/%s/migrations/%s\u0027 % (base.getid(server),"},{"line_number":81,"context_line":"                                           base.getid(migration)))"},{"line_number":82,"context_line":"        return self.convert_into_with_meta(body, resp)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9aed3d3a_b932460e","line":82,"range":{"start_line":79,"start_character":8,"end_line":82,"end_character":54},"updated":"2016-02-29 18:34:51.000000000","message":"Why not just self._delete?","commit_id":"3f08c23c47498765d45ca26f9a0a6e3cce1ce8b5"},{"author":{"_account_id":7461,"name":"Paul Murray","email":"pmurray@hpe.com","username":"ptm"},"change_message_id":"437354ccb4af8b4bf9e066760bf6fbbe5c635a1a","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        :param migration: Migration id that will be cancelled"},{"line_number":77,"context_line":"        :returns: An instance of novaclient.base.TupleWithMeta"},{"line_number":78,"context_line":"        \"\"\""},{"line_number":79,"context_line":"        resp, body \u003d self.api.client.delete("},{"line_number":80,"context_line":"            \u0027/servers/%s/migrations/%s\u0027 % (base.getid(server),"},{"line_number":81,"context_line":"                                           base.getid(migration)))"},{"line_number":82,"context_line":"        return self.convert_into_with_meta(body, resp)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9aed3d3a_dc8a7763","line":82,"range":{"start_line":79,"start_character":8,"end_line":82,"end_character":54},"in_reply_to":"9aed3d3a_960bba23","updated":"2016-03-01 08:37:33.000000000","message":"This is trivial and this patch had +W - a follow on would have been better at this stage","commit_id":"3f08c23c47498765d45ca26f9a0a6e3cce1ce8b5"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"3debfdb9ebb24b438fa033698d2c5c3e583d17ad","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        :param migration: Migration id that will be cancelled"},{"line_number":77,"context_line":"        :returns: An instance of novaclient.base.TupleWithMeta"},{"line_number":78,"context_line":"        \"\"\""},{"line_number":79,"context_line":"        resp, body \u003d self.api.client.delete("},{"line_number":80,"context_line":"            \u0027/servers/%s/migrations/%s\u0027 % (base.getid(server),"},{"line_number":81,"context_line":"                                           base.getid(migration)))"},{"line_number":82,"context_line":"        return self.convert_into_with_meta(body, resp)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9aed3d3a_960bba23","line":82,"range":{"start_line":79,"start_character":8,"end_line":82,"end_character":54},"in_reply_to":"9aed3d3a_b932460e","updated":"2016-03-01 02:30:07.000000000","message":"Done","commit_id":"3f08c23c47498765d45ca26f9a0a6e3cce1ce8b5"}]}
