Re: [PATCH] storage: revert bogus raw volumes support check

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

 




On 03/26/2016 01:10 PM, Roman Bogorodskiy wrote:
> Commits bb5f2dc and 6682d62 added a check like:
> 
>   if (vol->target.format != VIR_STORAGE_FILE_RAW)
> 
> to the CreateVol method for zfs and logical storage backends
> respectively. However, this check is bogus because
> virStorageVolDefParseXML() in conf/storage_conf.c sets target.format
> only if volOptions in virStoragePoolTypeInfo has formatFromString set,
> and that's not the case for zfs and logical backends. So the check
> always fails and breaks volume creation.
> 
> Thus, revert those commits.
> ---
>  src/storage/storage_backend_logical.c | 6 ------
>  src/storage/storage_backend_zfs.c     | 6 ------
>  2 files changed, 12 deletions(-)
> 

ACK - although I redid it slightly to use the git revert command and
adjusting the commit ...

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]