)]}'
{"smaug/tests/fullstack/test_restores.py":[{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"39799037dcdf05c3aba478d7a6991daf3592abf1","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        restore.create(self.provider_id, checkpoint.id,"},{"line_number":52,"context_line":"                       restore_target, self.parameters)"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        # volumes \u003d self.cinder_client.volumes.list()"},{"line_number":55,"context_line":"        # after_num \u003d len(volumes)"},{"line_number":56,"context_line":"        # self.assertEqual(1, after_num - before_num)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def test_restore_get(self):"},{"line_number":59,"context_line":"        volume \u003d self.store(objects.Volume())"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_6a8e407a","line":56,"range":{"start_line":54,"start_character":0,"end_line":56,"end_character":53},"updated":"2016-07-12 09:27:05.000000000","message":"Why do you comment this judgment statement?","commit_id":"b9435ab7f6d8e04f9afd33a6f507b3cf3a87a9d5"},{"author":{"_account_id":21648,"name":"zhangshuai","email":"446077695@qq.com","username":"zhangshuai"},"change_message_id":"63ab76a740d90130cf3ff5af26b8d192ff9b7d51","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        restore.create(self.provider_id, checkpoint.id,"},{"line_number":52,"context_line":"                       restore_target, self.parameters)"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        # volumes \u003d self.cinder_client.volumes.list()"},{"line_number":55,"context_line":"        # after_num \u003d len(volumes)"},{"line_number":56,"context_line":"        # self.assertEqual(1, after_num - before_num)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def test_restore_get(self):"},{"line_number":59,"context_line":"        volume \u003d self.store(objects.Volume())"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_12f658f9","line":56,"range":{"start_line":54,"start_character":0,"end_line":56,"end_character":53},"in_reply_to":"1aa78d24_6a8e407a","updated":"2016-07-14 03:03:06.000000000","message":"I told to you, when restore.create received \u0027success\u0027 status, self.cinder_client.volumes.list will run, but the background services hadn\u0027t create a new restoring volume,so after_num will be equal to before_num.\nSuggestion: add an field \u0027volume_id\u0027 to the return value of restore.create","commit_id":"b9435ab7f6d8e04f9afd33a6f507b3cf3a87a9d5"},{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"39799037dcdf05c3aba478d7a6991daf3592abf1","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        restore2 \u003d self.store(objects.Restore())"},{"line_number":90,"context_line":"        restore2.create(self.provider_id, checkpoint.id,"},{"line_number":91,"context_line":"                        restore_target, self.parameters)"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # restores \u003d self.smaug_client.restores.list()"},{"line_number":94,"context_line":"        # after_num \u003d len(restores)"},{"line_number":95,"context_line":"        # self.assertEqual(2, after_num - before_num)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_13009065","line":95,"range":{"start_line":92,"start_character":0,"end_line":95,"end_character":53},"updated":"2016-07-12 09:27:05.000000000","message":"Why do you comment this judgment statement?","commit_id":"b9435ab7f6d8e04f9afd33a6f507b3cf3a87a9d5"},{"author":{"_account_id":21648,"name":"zhangshuai","email":"446077695@qq.com","username":"zhangshuai"},"change_message_id":"63ab76a740d90130cf3ff5af26b8d192ff9b7d51","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        restore2 \u003d self.store(objects.Restore())"},{"line_number":90,"context_line":"        restore2.create(self.provider_id, checkpoint.id,"},{"line_number":91,"context_line":"                        restore_target, self.parameters)"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # restores \u003d self.smaug_client.restores.list()"},{"line_number":94,"context_line":"        # after_num \u003d len(restores)"},{"line_number":95,"context_line":"        # self.assertEqual(2, after_num - before_num)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_b2e5c439","line":95,"range":{"start_line":92,"start_character":0,"end_line":95,"end_character":53},"in_reply_to":"1aa78d24_13009065","updated":"2016-07-14 03:03:06.000000000","message":"ditto.","commit_id":"b9435ab7f6d8e04f9afd33a6f507b3cf3a87a9d5"}]}