)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"ec7e7a14f6608c192fc88e2ec7a560774a6c1aa3","unresolved":true,"context_lines":[{"line_number":750,"context_line":"        for instance in failed_eva_local_instances.values():"},{"line_number":751,"context_line":"            LOG.info(\u0027Destroying instance as it has been evacuated failed\u0027"},{"line_number":752,"context_line":"                     \u0027 and deleted from this host but still exists in the\u0027"},{"line_number":753,"context_line":"                     \u0027 hypervisor\u0027,"},{"line_number":754,"context_line":"                     instance\u003dinstance)"},{"line_number":755,"context_line":"            try:"},{"line_number":756,"context_line":"                instance \u003d objects.Instance.get_by_uuid("}],"source_content_type":"text/x-python","patch_set":2,"id":"5d2c51da_c504f3c0","line":753,"updated":"2021-10-27 14:18:28.000000000","message":"Hum, no. If we delete the instance now, given the first evacuation was having a exception, then there are no recreated instance somewhere else.\n\nMaybe it\u0027s me not understanding, but I think we would delete some pet instance here when restarting even if the operator doesn\u0027t want.","commit_id":"63d4143e84913930622666d87510c399590e1779"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"f8f086e36ec4259493072aaed51962b7e09a9ab9","unresolved":true,"context_lines":[{"line_number":750,"context_line":"        for instance in failed_eva_local_instances.values():"},{"line_number":751,"context_line":"            LOG.info(\u0027Destroying instance as it has been evacuated failed\u0027"},{"line_number":752,"context_line":"                     \u0027 and deleted from this host but still exists in the\u0027"},{"line_number":753,"context_line":"                     \u0027 hypervisor\u0027,"},{"line_number":754,"context_line":"                     instance\u003dinstance)"},{"line_number":755,"context_line":"            try:"},{"line_number":756,"context_line":"                instance \u003d objects.Instance.get_by_uuid("}],"source_content_type":"text/x-python","patch_set":2,"id":"e0ea3a18_99adf8fc","line":753,"in_reply_to":"5d2c51da_c504f3c0","updated":"2021-11-03 09:32:11.000000000","message":"for the instances that evacuate failed and deleted in database, we should destroy them on the hypervisior.\nL756 checks the exist of the instance.","commit_id":"63d4143e84913930622666d87510c399590e1779"}]}
