Re: [Autotest] [PATCH 1/2] KVM Test: Update cmd() help function in kvm_monitor.py to support parameters.

2010-07-12 Thread Lucas Meneghel Rodrigues
On Mon, 2010-06-28 at 05:43 -0400, Feng Yang wrote: - Michael Goldish mgold...@redhat.com wrote: Why not add a wrapper for the command you're interested in? If we do it your way, a test that uses cmd() with parameters will have to handle the human case and the QMP case separately.

Re: [Autotest] [PATCH 1/2] KVM Test: Update cmd() help function in kvm_monitor.py to support parameters.

2010-07-12 Thread Michael Goldish
On 07/12/2010 10:57 PM, Lucas Meneghel Rodrigues wrote: On Mon, 2010-06-28 at 05:43 -0400, Feng Yang wrote: - Michael Goldish mgold...@redhat.com wrote: Why not add a wrapper for the command you're interested in? If we do it your way, a test that uses cmd() with parameters will have to

Re: [Autotest] [PATCH 1/2] KVM Test: Update cmd() help function in kvm_monitor.py to support parameters.

2010-06-28 Thread Michael Goldish
On 06/28/2010 10:45 AM, Feng Yang wrote: This function allow user send qmp command with parameters. e.g. balloon value=1073741824 Also log command to debug. Signed-off-by: Feng Yang fy...@redhat.com --- client/tests/kvm/kvm_monitor.py | 32 1 files

Re: [Autotest] [PATCH 1/2] KVM Test: Update cmd() help function in kvm_monitor.py to support parameters.

2010-06-28 Thread Feng Yang
] [PATCH 1/2] KVM Test: Update cmd() help function in kvm_monitor.py to support parameters. On 06/28/2010 10:45 AM, Feng Yang wrote: This function allow user send qmp command with parameters. e.g. balloon value=1073741824 Also log command to debug. Signed-off-by: Feng Yang fy