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

Re: [libvirt] [PATCH 1/6] list: Define new API virStoragePoolListAllVolumes



On 09/07/2012 07:28 AM, Peter Krempa wrote:
> On 09/04/12 17:32, Osier Yang wrote:
>> Simply returns the storage volume objects. No supported filter
>> flags.
>>
>> include/libvirt/libvirt.h.in: Declare the API
>> python/generator.py: Skip the function for generating. virStoragePool.py
>>                       will be added in later patch.
>> src/driver.h: virDrvStoragePoolListVolumesFlags
>> src/libvirt.c: Implementation for the API.
>> src/libvirt_public.syms: Export the symbol to public
>> ---
>>   include/libvirt/libvirt.h.in |    3 ++
>>   python/generator.py          |    1 +
>>   src/driver.h                 |    6 ++++-
>>   src/libvirt.c                |   50
>> ++++++++++++++++++++++++++++++++++++++++++
>>   src/libvirt_public.syms      |    1 +
>>   5 files changed, 60 insertions(+), 1 deletions(-)
>>
> 
> Maybe there are some properties that we could use to filter storage
> volumes, but those can also be added later.

Agreed that additional filters can be added later.  Off the top of my
head, I can think of several useful filters (and I'm pretty sure not all
of these filter idears are even exposed by current libvirt API, so we
would have to add other API before we could even filter on these bits):
whether the storage volume is encrypted, whether it is compressed
(possibly even by which compression method), whether the storage volume
is sparse or fully allocated, whether the storage volume has a backing
file or is standalone, whether the storage volume has internal
snapshots, current lock manager status (unlocked, shared lock,
read-write lock).

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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