Re: [PATCH] virt.virt_env_process: Abstract screenshot production

2011-09-26 Thread Lukáš Doktor
Hi, vm.screendump() doesn't have parameter 'debug'. So you should either add debug parameter to kvm_vm.py or remove this parameter (and perhaps add debug=False into kvm_vm.py). Regards, Lukáš Dne 24.9.2011 01:27, Lucas Meneghel Rodrigues napsal(a): In order to ease work with other

Re: [PATCH] virt.virt_env_process: Abstract screenshot production

2011-09-26 Thread Lucas Meneghel Rodrigues
On 09/26/2011 09:27 AM, Lukáš Doktor wrote: Hi, vm.screendump() doesn't have parameter 'debug'. My fault, the screendump method on both qmp and human monitors does take this parameter, and since the implementation on virt_env_process was using the monitor method directly, I forgot to add

Re: [PATCH] virt.virt_env_process: Abstract screenshot production

2011-09-26 Thread Lukáš Doktor
Dne 26.9.2011 15:10, Lucas Meneghel Rodrigues napsal(a): On 09/26/2011 09:27 AM, Lukáš Doktor wrote: Hi, vm.screendump() doesn't have parameter 'debug'. My fault, the screendump method on both qmp and human monitors does take this parameter, and since the implementation on virt_env_process

Re: [PATCH] virt.virt_env_process: Abstract screenshot production

2011-09-26 Thread Lucas Meneghel Rodrigues
On 09/26/2011 10:20 AM, Lukáš Doktor wrote: Dne 26.9.2011 15:10, Lucas Meneghel Rodrigues napsal(a): On 09/26/2011 09:27 AM, Lukáš Doktor wrote: Hi, vm.screendump() doesn't have parameter 'debug'. My fault, the screendump method on both qmp and human monitors does take this parameter, and

[PATCH] virt.virt_env_process: Abstract screenshot production

2011-09-23 Thread Lucas Meneghel Rodrigues
In order to ease work with other virtualization types, make virt_env_process to call vm.screendump() instead of vm.monitor.screendump(). Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/virt/virt_env_process.py |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-)