This patch makes the env processor code that takes a
VM screenshot pre/post a test runs depending on the
value of the take_regular_screendumps config variable.
Usually, we either want all screenshots or none of them.
Signed-off-by: Lucas Meneghel Rodrigues <lmr@xxxxxxxxxx>
---
client/virt/virt_env_process.py | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/client/virt/virt_env_process.py b/client/virt/virt_env_process.py
index 8ef6497..8ab9e3d 100644
--- a/client/virt/virt_env_process.py
+++ b/client/virt/virt_env_process.py
@@ -86,7 +86,7 @@ def preprocess_vm(test, params, env, name):
scrdump_filename = os.path.join(test.debugdir, "pre_%s.ppm" % name)
try:
- if vm.monitor:
+ if vm.monitor and params.get("take_regular_screendumps") == "yes":
vm.monitor.screendump(scrdump_filename, debug=False)
except kvm_monitor.MonitorError, e:
logging.warn(e)
@@ -122,7 +122,7 @@ def postprocess_vm(test, params, env, name):
scrdump_filename = os.path.join(test.debugdir, "post_%s.ppm" % name)
try:
- if vm.monitor:
+ if vm.monitor and params.get("take_regular_screenshots") == "yes":
vm.monitor.screendump(scrdump_filename, debug=False)
except kvm_monitor.MonitorError, e:
logging.warn(e)
--
1.7.6
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
