[libvirt] [PATCH 1/4] snapshot: add two more filter sets to API

Eric Blake eblake at redhat.com
Wed Nov 14 18:33:05 UTC 2012


On 11/13/2012 04:50 PM, Peter Krempa wrote:
> On 11/13/12 20:09, Eric Blake wrote:
>> As we enable more modes of snapshot creation, it becomes more important
>> to be able to quickly filter based on snapshot properties.  This patch
>> introduces new filter flags; subsequent patches will introduce virsh
>> back-compat filtering, as well as actual libvirt filtering.
>>

>> +++ b/include/libvirt/libvirt.h.in
>> @@ -3818,6 +3818,25 @@ typedef enum {
>>                                                           which have
>> metadata */
>>       VIR_DOMAIN_SNAPSHOT_LIST_NO_METADATA = (1 << 4), /* Filter by
>> snapshots
>>                                                           with no
>> metadata */
>> +
>> +    VIR_DOMAIN_SNAPSHOT_LIST_OFFLINE     = (1 << 5), /* Filter by
>> snapshots
>> +                                                        taken while
>> guest was
>> +                                                        offline */
>> +    VIR_DOMAIN_SNAPSHOT_LIST_ONLINE      = (1 << 6), /* Filter by
>> snapshots
>> +                                                        taken while
>> guest was
>> +                                                        online, and with
>> +                                                        memory state */
> 
> I'm wondering if we shouldn't go for ACTIVE/INACTIVE here?

Indeed, that is more consistent with what we have used elsewhere.  I'll
respin the series with that change.

> Otherwise looks okay so I'm comfortable with ACKing this when the naming
> question gets settled.

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 617 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20121114/55a0341c/attachment-0001.sig>


More information about the libvir-list mailing list