[libvirt] [PATCH 07/10] tests: qemucapsprobemock: Prepare 'qemucapsprobe' for the new format

John Ferlan jferlan at redhat.com
Fri Jun 8 01:58:09 UTC 2018



On 06/04/2018 09:58 AM, Peter Krempa wrote:
> Change the output of qemucapsprobe to record the commands used for
> querying. This allows to easily identify which reply belongs to which
> command and also will allow to test whether we use stable queries.

Yay!  So painful to try and matching things up.

> 
> This change includes changing dropping of the QMP greeting from the file
> and reformatting of the query and output to stdout.
> 
> Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> ---
>  tests/qemucapsprobemock.c | 31 ++++++++++++++++++++++---------
>  1 file changed, 22 insertions(+), 9 deletions(-)
> 

Looks like there's two different things in this patch, but maybe I'm
wrong too.

The qemuMonitorJSONIOProcessLine changes seem to be separable, true?
Although the bool first move out has me slightly doubting...

Then the qemuMonitorSend does what the first commit paragraph states, right?

Doesn't the greeting get dropped in the old code too or am I missing
something subtle this late at night?

The code is fine - your call on the separation.

Reviewed-by: John Ferlan <jferlan at redhat.com>

John




More information about the libvir-list mailing list