Re: [PATCH 33/35] tests: qemublock: Test handling of 'unmap' and 'detect-zeroes' options

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 04/25/2018 11:15 AM, Peter Krempa wrote:
> The test cases would correspond to the following -drive command lines:
> 
> file-backing_basic-detect.xml:
> -drive file=/var/lib/libvirt/images/a,format=qcow,if=none,id=drive-dummy,detect-zeroes=on
> -device virtio-blk-pci,scsi=off,drive=drive-dummy,id=dummy
> 
> file-backing_basic-unmap-detect.xml:
> -drive file=/var/lib/libvirt/images/a,format=qcow,if=none,id=drive-dummy,discard=unmap,detect-zeroes=unmap
> -device virtio-blk-pci,scsi=off,drive=drive-dummy,id=dummy
> 
> file-backing_basic-unmap-ignore.xml:
> -drive file=/var/lib/libvirt/images/a,format=qcow,if=none,id=drive-dummy,discard=ignore,detect-zeroes=on
> -device virtio-blk-pci,scsi=off,drive=drive-dummy,id=dummy
> 
> file-backing_basic-unmap.xml:
> -drive file=/var/lib/libvirt/images/a,format=qcow,if=none,id=drive-dummy,discard=unmap
> -device virtio-blk-pci,scsi=off,drive=drive-dummy,id=dummy
> ---
>  tests/qemublocktest.c                              |  5 ++
>  .../xml2json/file-backing_basic-detect.json        | 60 ++++++++++++++++++++
>  .../xml2json/file-backing_basic-detect.xml         | 47 ++++++++++++++++
>  .../xml2json/file-backing_basic-unmap-detect.json  | 64 ++++++++++++++++++++++
>  .../xml2json/file-backing_basic-unmap-detect.xml   | 47 ++++++++++++++++
>  .../xml2json/file-backing_basic-unmap-discard.json |  0

^^  You have a stray

>  .../xml2json/file-backing_basic-unmap-ignore.json  | 64 ++++++++++++++++++++++
>  .../xml2json/file-backing_basic-unmap-ignore.xml   | 47 ++++++++++++++++
>  .../xml2json/file-backing_basic-unmap.json         | 63 +++++++++++++++++++++
>  .../xml2json/file-backing_basic-unmap.xml          | 47 ++++++++++++++++
>  10 files changed, 444 insertions(+)
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-detect.json
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-detect.xml
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-unmap-detect.json
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-unmap-detect.xml
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-unmap-discard.json
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-unmap-ignore.json
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-unmap-ignore.xml
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-unmap.json
>  create mode 100644 tests/qemublocktestdata/xml2json/file-backing_basic-unmap.xml
> 

Discarding the stray... (sorry)...

Yet another case where there's trust that you have the right output here...

Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>

John

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux