[libvirt] [PATCH 20/24] tests: qemumonitorjson: Fix 'gluster' node name detection test case

[Eric Blake]

On 07/26/2017 05:00 AM, Peter Krempa wrote:
> Add the blockstats data and fix the expected output.
> ---
>  ...emumonitorjson-nodename-gluster-blockstats.json | 111 +++++++++++++++++++++
>  ...mumonitorjson-nodename-gluster-named-nodes.json |  90 ++++++++---------
>  .../qemumonitorjson-nodename-gluster.result        |  15 ++-
>  tests/qemumonitorjsontest.c                        |   2 +-
>  4 files changed, 167 insertions(+), 51 deletions(-)
>  create mode 100644 tests/qemumonitorjsondata/qemumonitorjson-nodename-gluster-blockstats.json

Reviewed-by: Eric Blake <eblake at redhat.com>

> 
> diff --git a/tests/qemumonitorjsondata/qemumonitorjson-nodename-gluster-blockstats.json b/tests/qemumonitorjsondata/qemumonitorjson-nodename-gluster-blockstats.json
> new file mode 100644
> index 000000000..665a6b05b
> --- /dev/null
> +++ b/tests/qemumonitorjsondata/qemumonitorjson-nodename-gluster-blockstats.json
> @@ -0,0 +1,111 @@
> +[
> +    {
> +      "device": "drive-virtio-disk0",
> +      "parent": {
> +        "stats": {
> +          "flush_total_time_ns": 0,

This output was obviously generated by loading a simple <domain> XML and
then getting query-blockstats output.  I'm wondering, if for
reproducibility (modulo the change of random nodename suffixes), you
should enhance the commit message here (and in similar other patches) to
call out the steps you used to create the domain (did you have to use
qemu-img, and what <disk> spec did you use).  It's not a hard request,
as I've already given reviews, but it may be something we appreciate in
the future when doing git archaeology.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170726/e6328bf3/attachment-0001.sig>