From: Michael Goldish <mgold...@redhat.com>
Print outputs (before and after migration) of migration_test_command only if
they differ. Currently we print them anyway, and the outputs can be very long.
Signed-off-by: Michael Goldish <mgold...@redhat.com>
diff --git a/client/tests/kvm_runtest_2/kvm_tests.py
b/client/tests/kvm_runtest_2/kvm_tests.py
index 9733dc1..7349524 100644
--- a/client/tests/kvm_runtest_2/kvm_tests.py
+++ b/client/tests/kvm_runtest_2/kvm_tests.py
@@ -80,8 +80,6 @@ def run_migration(test, params, env):
reference_output =
session.get_command_output(params.get("migration_test_command"))
session.close()
- kvm_log.info("Reference output of command '%s':\n%s" %
(params.get("migration_test_command"), reference_output))
-
# Define the migration command
cmd = "migrate -d tcp:localhost:%d" % dest_vm.migration_port
kvm_log.debug("Migration command: %s" % cmd)
@@ -162,11 +160,13 @@ def run_migration(test, params, env):
output = session.get_command_output(params.get("migration_test_command"))
session.close()
- kvm_log.info("Output of command '%s' after migration:\n%s" %
(params.get("migration_test_command"), output))
-
# Compare output to reference output
if output != reference_output:
- raise error.TestFail, "Command output before and after migration
differs"
+ kvm_log.info("Command output before migration differs from command
output after migration")
+ kvm_log.info("Command: %s" % params.get("migration_test_command"))
+ kvm_log.info("Output before: %s" % reference_output)
+ kvm_log.info("Output after: %s" % output)
+ raise error.TestFail, "Command produced different output before and
after migration"
def run_autotest(test, params, env):
--
To unsubscribe from this list: send the line "unsubscribe kvm-commits" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
