This is an example of a case where context information is useful.
Signed-off-by: Michael Goldish <[email protected]>
---
.../kvm/tests/migration_with_file_transfer.py | 8 +++++---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/client/tests/kvm/tests/migration_with_file_transfer.py
b/client/tests/kvm/tests/migration_with_file_transfer.py
index 27c1fe1..d0159c5 100644
--- a/client/tests/kvm/tests/migration_with_file_transfer.py
+++ b/client/tests/kvm/tests/migration_with_file_transfer.py
@@ -47,24 +47,26 @@ def run_migration_with_file_transfer(test, params, env):
finally:
bg.join()
- logging.info("Transferring file from host to guest")
+ error.context("transferring file to guest while migrating")
bg = kvm_utils.Thread(vm.copy_files_to,
(host_path, guest_path, 0, transfer_timeout))
run_and_migrate(bg)
- logging.info("Transferring file back from guest to host")
+ error.context("transferring file back to host while migrating")
bg = kvm_utils.Thread(vm.copy_files_from,
(guest_path, host_path_returned, 0,
transfer_timeout))
run_and_migrate(bg)
- # Make sure the returned file is indentical to the original one
+ # Make sure the returned file is identical to the original one
+ error.context("comparing hashes")
orig_hash = client_utils.hash_file(host_path)
returned_hash = client_utils.hash_file(host_path_returned)
if orig_hash != returned_hash:
raise error.TestFail("Returned file hash (%s) differs from "
"original one (%s)" % (returned_hash,
orig_hash))
+ error.context("cleanup")
finally:
session.close()
if os.path.isfile(host_path):
--
1.7.3.4
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
