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 <[email protected]> --- 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 _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
