Re: [PATCH 01/19] storage: Adjust qemu-img switches check

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

 



On Mon, Jun 13, 2016 at 20:27:40 -0400, John Ferlan wrote:
> Since we support QEMU 0.12 and later, checking for support of specific flags
> added prior to that isn't necessary.
> 
> Thus start with the base of having the "-o options" available for the
> qemu-img create option and then determine whether we have the compat
> option for qcow2 files (which would be necessary up through qemu 2.0
> where the default changes to compat 0.11).
> 
> Adjust test to no long check for NONE and FLAG options as well was removing
> results of tests that would use that option.
> 
> Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
> ---
>  src/storage/storage_backend.c                      | 91 +++++++---------------
>  tests/storagevolxml2argvdata/qcow2-flag.argv       |  2 -
>  .../qcow2-nobacking-convert-flag.argv              |  2 -
>  .../qcow2-nobacking-convert-none.argv              |  2 -
>  .../qcow2-nobacking-flag.argv                      |  1 -
>  .../qcow2-nobacking-none.argv                      |  1 -
>  tests/storagevolxml2argvdata/qcow2-none.argv       |  1 -
>  tests/storagevolxml2argvtest.c                     | 22 +-----
>  8 files changed, 29 insertions(+), 93 deletions(-)
>  delete mode 100644 tests/storagevolxml2argvdata/qcow2-flag.argv
>  delete mode 100644 tests/storagevolxml2argvdata/qcow2-nobacking-convert-flag.argv
>  delete mode 100644 tests/storagevolxml2argvdata/qcow2-nobacking-convert-none.argv
>  delete mode 100644 tests/storagevolxml2argvdata/qcow2-nobacking-flag.argv
>  delete mode 100644 tests/storagevolxml2argvdata/qcow2-nobacking-none.argv
>  delete mode 100644 tests/storagevolxml2argvdata/qcow2-none.argv

I still think the code should be refactored sooner or later, but ACK.

--
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]