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@xxxxxxxxxx +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list