[libvirt] [PATCH v2 4/6] vol: Add thin_pool to _virStorageVolSource
John Ferlan
jferlan at redhat.com
Mon Feb 1 20:52:12 UTC 2016
>> <volume type='block'>
>> ...
>> <source>
>> <name>%s</name>
>> <format type='thin_pool'>
>> </source>
>> ...
>> </volume>
>>
>> Trying to forward think if/when a vol.xml was used as input...
>
> Maybe we should stop right here and handle the thin pool/volumes in a separate
> patch series. It would be best to create a new libvirt pool type that will be
> dedicated for lvm thin pools and add a proper support for thin pools.
>
>
Without patches 4 & 5 we get the following for virsh vol-dumpxml:
<volume ...>
...
<source>
</source>
...
</volume>
Which is OK - although it looks a bit strange to me - it isn't the only
output that doesn't have <source> defined.
Since a thin lv and thin-pool can be placed into any vg and a vg is
essentially a libvirt pool, I'm not so sure having a dedicated pool type
is necessary. Sure it'd make some things easier, but can we assume every
thin volume instance is configured that way?
John
More information about the libvir-list
mailing list