)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1fbd24c91526f67df34b84098057a97d21d5da72","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When there is an error in the compute manager relating to"},{"line_number":10,"context_line":"live-migrate, this change ensures that an instance fault is"},{"line_number":11,"context_line":"recorded to help debug and issues."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"We also need to look at better reporting of errors to users,"},{"line_number":14,"context_line":"but this is not in the scope of this small change."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATn%2F88yI%3D","line":11,"updated":"2013-12-18 13:45:36.000000000","message":"s/and/any/","commit_id":"3b6985801e32723207418a6fe90cc831f67e61b1"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0ba7996bacb7b89639ec75e395b77674eb69487f","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When there is an error in the compute manager relating to"},{"line_number":10,"context_line":"live-migrate, this change ensures that an instance fault is"},{"line_number":11,"context_line":"recorded to help debug and issues."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"We also need to look at better reporting of errors to users,"},{"line_number":14,"context_line":"but this is not in the scope of this small change."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAT3%2F%2Fr9E%3D","line":11,"in_reply_to":"AAAATn%2F88yI%3D","updated":"2014-01-03 13:57:09.000000000","message":"Done","commit_id":"3b6985801e32723207418a6fe90cc831f67e61b1"}],"nova/tests/compute/test_compute_mgr.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"575e1bbc4e5f03a906d41239b461887d28098ad6","unresolved":false,"context_lines":[{"line_number":797,"context_line":"        self._test_check_can_live_migrate_destination(has_mig_data\u003dTrue)"},{"line_number":798,"context_line":""},{"line_number":799,"context_line":"    def test_check_can_live_migrate_destination_fail(self):"},{"line_number":800,"context_line":"        self.mox.StubOutWithMock(self.compute.conductor_api,"},{"line_number":801,"context_line":"                                 \u0027instance_fault_create\u0027)"},{"line_number":802,"context_line":"        self.compute.conductor_api.instance_fault_create(self.context,"},{"line_number":803,"context_line":"                mox.IgnoreArg())"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAT3%2F%2FqhQ%3D","line":800,"updated":"2014-01-03 21:18:53.000000000","message":"Shouldn\u0027t these mocks live up in _test_check_can_live_migrate_destination if do_raise is True?","commit_id":"775818276b15f2d68271ad020e865baafad5df1a"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"833cd7ae8971adefce0bcf0b71f693e27ad34b27","unresolved":false,"context_lines":[{"line_number":797,"context_line":"        self._test_check_can_live_migrate_destination(has_mig_data\u003dTrue)"},{"line_number":798,"context_line":""},{"line_number":799,"context_line":"    def test_check_can_live_migrate_destination_fail(self):"},{"line_number":800,"context_line":"        self.mox.StubOutWithMock(self.compute.conductor_api,"},{"line_number":801,"context_line":"                                 \u0027instance_fault_create\u0027)"},{"line_number":802,"context_line":"        self.compute.conductor_api.instance_fault_create(self.context,"},{"line_number":803,"context_line":"                mox.IgnoreArg())"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUH%2F%2F7IQ%3D","line":800,"in_reply_to":"AAAAT3%2F%2FqhQ%3D","updated":"2014-01-06 18:26:20.000000000","message":"Yeah, they probably should. I moved it here to stop making the other test more complex, but I guess it actually makes it broken.","commit_id":"775818276b15f2d68271ad020e865baafad5df1a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b18274f2de65c988ffdc541a4ad917684cd91e54","unresolved":false,"context_lines":[{"line_number":797,"context_line":"        self._test_check_can_live_migrate_destination(has_mig_data\u003dTrue)"},{"line_number":798,"context_line":""},{"line_number":799,"context_line":"    def test_check_can_live_migrate_destination_fail(self):"},{"line_number":800,"context_line":"        self.mox.StubOutWithMock(self.compute.conductor_api,"},{"line_number":801,"context_line":"                                 \u0027instance_fault_create\u0027)"},{"line_number":802,"context_line":"        self.compute.conductor_api.instance_fault_create(self.context,"},{"line_number":803,"context_line":"                mox.IgnoreArg())"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUH%2F%2Fxd0%3D","line":800,"in_reply_to":"AAAAUH%2F%2F7IQ%3D","updated":"2014-01-08 16:36:50.000000000","message":"Yeah, if another test comes alone and passes do_raise\u003dTrue to _test_check_can_live_migrate_destination this is busted for them also, so it should live in the common method.","commit_id":"775818276b15f2d68271ad020e865baafad5df1a"}]}