From 57a8c5e7fdf2e2542fb95c00c5e5ae2b45a43fa0 Mon Sep 17 00:00:00 2001 From: Matt Riedemann Date: Fri, 7 Jul 2017 02:15:42 -0400 Subject: [PATCH] Remove reverts_task_state decorator from swap_volume We never change the instance task_state during a swap volume operation, so there is no reason to use the reverts_task_state decorator, as there is nothing to revert back to in case of a failure. Change-Id: I437ecb16682d69f83e6ec4fe0ca54fcc168ff448 --- nova/compute/manager.py | 1 - 1 file changed, 1 deletion(-) diff --git a/nova/compute/manager.py b/nova/compute/manager.py index 73dd64b8a7d1..41d9d6354e33 100644 --- a/nova/compute/manager.py +++ b/nova/compute/manager.py @@ -5098,7 +5098,6 @@ class ComputeManager(manager.Manager): return (comp_ret, new_cinfo) @wrap_exception() - @reverts_task_state @wrap_instance_fault def swap_volume(self, context, old_volume_id, new_volume_id, instance): """Swap volume for an instance."""