The code to remove the migration files was one indentation
block deeper than it should - my mistake. Now, we ensure
the files are being cleaned up at the end of the test.

Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
---
 client/tests/kvm/kvm_test_utils.py |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/client/tests/kvm/kvm_test_utils.py 
b/client/tests/kvm/kvm_test_utils.py
index da33b8b..53c11ae 100644
--- a/client/tests/kvm/kvm_test_utils.py
+++ b/client/tests/kvm/kvm_test_utils.py
@@ -222,8 +222,10 @@ def migrate(vm, env=None, mig_timeout=3600, 
mig_protocol="tcp",
         if "paused" in o:
             logging.debug("Destination VM is paused, resuming it...")
             dest_vm.monitor.cmd("cont")
-            if os.path.exists(migration_file):
-                os.remove(migration_file)
+
+        if os.path.exists(migration_file):
+            logging.debug("Removing migration file %s", migration_file)
+            os.remove(migration_file)
 
         # Kill the source VM
         vm.destroy(gracefully=False)
-- 
1.7.0.1

_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to