)]}'
{"nova/compute/api.py":[{"author":{"_account_id":17814,"name":"fengzhr","email":"fengzhr@awcloud.com","username":"space"},"change_message_id":"336e7cb592a2580a13e42d4d210aa828385d400c","unresolved":false,"context_lines":[{"line_number":1801,"context_line":"                                instance.destroy()"},{"line_number":1802,"context_line":"                    else:"},{"line_number":1803,"context_line":"                        instance.destroy()"},{"line_number":1804,"context_line":"                    quotas.commit()"},{"line_number":1805,"context_line":"            except exception.InstanceNotFound:"},{"line_number":1806,"context_line":"                quotas.rollback()"},{"line_number":1807,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"dfeb2761_7a474b4d","line":1804,"range":{"start_line":1804,"start_character":20,"end_line":1804,"end_character":26},"updated":"2017-04-01 02:12:25.000000000","message":"If instance is None, this won\u0027t be executed. And then we couldn\u0027t find the instance, but usage still exist.","commit_id":"5c1af55cbe526dea72308767df8709064ffae6a8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"279e932ee0c116fe11d267926e9261613dffe042","unresolved":false,"context_lines":[{"line_number":1801,"context_line":"                                instance.destroy()"},{"line_number":1802,"context_line":"                    else:"},{"line_number":1803,"context_line":"                        instance.destroy()"},{"line_number":1804,"context_line":"                    quotas.commit()"},{"line_number":1805,"context_line":"            except exception.InstanceNotFound:"},{"line_number":1806,"context_line":"                quotas.rollback()"},{"line_number":1807,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"dfeb2761_bae39305","line":1804,"range":{"start_line":1804,"start_character":20,"end_line":1804,"end_character":26},"in_reply_to":"dfeb2761_7a474b4d","updated":"2017-04-01 02:31:31.000000000","message":"Hmm, good point. Then we probably need an else branch where we rollback the reservation if instance is None.","commit_id":"5c1af55cbe526dea72308767df8709064ffae6a8"}]}