)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12299,"name":"Paweł Koniszewski","email":"pawel.koniszewski@gmail.com","username":"pawel.koniszewski"},"change_message_id":"d52e1df3ba2a75cffe99593888962a5de0e99c28","unresolved":false,"context_lines":[{"line_number":10,"context_line":"and update database record in post_live_migration_at_destination."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I146a66eff21ddbc396f90997a4aa728cbef61ad4"},{"line_number":13,"context_line":"Closes-Bug: #1452568"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"da6ed579_7aa93c23","line":13,"updated":"2016-01-15 14:10:21.000000000","message":"This should be removed because it does not fix this issue","commit_id":"ec9e24c62f6013cbcaf24f459dd6241308ed7f73"}],"nova/compute/manager.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"25a496f6fc6c469a8999188ada53c84b20438d01","unresolved":false,"context_lines":[{"line_number":5391,"context_line":"            LOG.exception(_LE(\u0027Failed to get compute_info for %s\u0027), self.host)"},{"line_number":5392,"context_line":"        finally:"},{"line_number":5393,"context_line":"            instance_az \u003d az.get_host_availability_zone(context, self.host)"},{"line_number":5394,"context_line":"            instance.availability_zone \u003d instance_az"},{"line_number":5395,"context_line":"            instance.host \u003d self.host"},{"line_number":5396,"context_line":"            instance.power_state \u003d current_power_state"},{"line_number":5397,"context_line":"            instance.task_state \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_27e9d680","line":5394,"updated":"2015-12-07 08:28:48.000000000","message":"can you please add in a unit test for this","commit_id":"c587f2b1a9101b83a7fef15dbe070b731019df48"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"280b41ac5691ed165ba1ec944d2ac2ca08ec893e","unresolved":false,"context_lines":[{"line_number":5391,"context_line":"            LOG.exception(_LE(\u0027Failed to get compute_info for %s\u0027), self.host)"},{"line_number":5392,"context_line":"        finally:"},{"line_number":5393,"context_line":"            instance_az \u003d az.get_host_availability_zone(context, self.host)"},{"line_number":5394,"context_line":"            instance.availability_zone \u003d instance_az"},{"line_number":5395,"context_line":"            instance.host \u003d self.host"},{"line_number":5396,"context_line":"            instance.power_state \u003d current_power_state"},{"line_number":5397,"context_line":"            instance.task_state \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_1de4f8de","line":5394,"updated":"2015-12-08 03:31:57.000000000","message":"why not just put these in a single line(without the temp veriable).","commit_id":"c587f2b1a9101b83a7fef15dbe070b731019df48"}],"nova/tests/unit/compute/test_compute_mgr.py":[{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"46d7eeb61d2e71349850b142cf1448b096febebc","unresolved":false,"context_lines":[{"line_number":4469,"context_line":"                                                     self.instance)"},{"line_number":4470,"context_line":"            _get_compute_info.assert_called_once_with(self.context,"},{"line_number":4471,"context_line":"                                                      self.compute.host)"},{"line_number":4472,"context_line":""},{"line_number":4473,"context_line":"            self.assertEqual(self.compute.host, self.instance.host)"},{"line_number":4474,"context_line":"            self.assertEqual(\u0027test_host\u0027, self.instance.node)"},{"line_number":4475,"context_line":"            self.assertEqual(\u0027fakezone\u0027, self.instance.availability_zone)"}],"source_content_type":"text/x-python","patch_set":10,"id":"7af24918_3d2f79f7","line":4472,"updated":"2016-03-02 19:23:34.000000000","message":"nit: please add the check to see if _get_compute_zone() is being called.\n_get_compute_zone.assert_called_once_with(self.context,self.compute.host)","commit_id":"d236f031d7234f00e54ae966d040bdef99d09910"},{"author":{"_account_id":12299,"name":"Paweł Koniszewski","email":"pawel.koniszewski@gmail.com","username":"pawel.koniszewski"},"change_message_id":"b0b20e2b4bbbb8fe1ca0f91a913419ce7b69d641","unresolved":false,"context_lines":[{"line_number":4469,"context_line":"                                                     self.instance)"},{"line_number":4470,"context_line":"            _get_compute_info.assert_called_once_with(self.context,"},{"line_number":4471,"context_line":"                                                      self.compute.host)"},{"line_number":4472,"context_line":""},{"line_number":4473,"context_line":"            self.assertEqual(self.compute.host, self.instance.host)"},{"line_number":4474,"context_line":"            self.assertEqual(\u0027test_host\u0027, self.instance.node)"},{"line_number":4475,"context_line":"            self.assertEqual(\u0027fakezone\u0027, self.instance.availability_zone)"}],"source_content_type":"text/x-python","patch_set":10,"id":"7af24918_5264ef20","line":4472,"in_reply_to":"7af24918_3d2f79f7","updated":"2016-03-02 19:39:50.000000000","message":"availability zone would not be set if it is not called.","commit_id":"d236f031d7234f00e54ae966d040bdef99d09910"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"8ff861de8d16cdfeb846439164cf383d770c987a","unresolved":false,"context_lines":[{"line_number":4469,"context_line":"                                                     self.instance)"},{"line_number":4470,"context_line":"            _get_compute_info.assert_called_once_with(self.context,"},{"line_number":4471,"context_line":"                                                      self.compute.host)"},{"line_number":4472,"context_line":""},{"line_number":4473,"context_line":"            self.assertEqual(self.compute.host, self.instance.host)"},{"line_number":4474,"context_line":"            self.assertEqual(\u0027test_host\u0027, self.instance.node)"},{"line_number":4475,"context_line":"            self.assertEqual(\u0027fakezone\u0027, self.instance.availability_zone)"}],"source_content_type":"text/x-python","patch_set":10,"id":"7af24918_5e9d1858","line":4472,"in_reply_to":"7af24918_5264ef20","updated":"2016-03-02 20:53:45.000000000","message":"yes.that\u0027s true.But I thought it would be good to check if get_compute_zone is being called with right arguments. Like it was done for get_power_state and get_compute_info.","commit_id":"d236f031d7234f00e54ae966d040bdef99d09910"}]}