Re: [PATCH v7 06/23] backup: Introduce virDomainCheckpoint APIs

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

 



On 3/27/19 6:16 AM, Daniel P. Berrangé wrote:
> On Wed, Mar 27, 2019 at 05:10:37AM -0500, Eric Blake wrote:
>> Introduce a bunch of new public APIs related to backup checkpoints.
>> Checkpoints are modeled heavily after virDomainSnapshotPtr (both
>> represent a point in time of the guest), although a snapshot exists
>> with the intent of rolling back to that state, while a checkpoint
>> exists to make it possible to create an incremental backup at a later
>> time.
>>

>> Operate on a domain object to list all children:
>> virDomainSnapshotNum                (no counterpart, this is the old
>> virDomainSnapshotListNames           racy interface)
>> virDomainSnapshotListAllSnapshots   virDomainListCheckpoints
> 
> s/virDomainListCheckpoints/virDomainListAllCheckpoints/

I was so busy fixing it in the code, I forgot the commit message.


> With the commit message fixed
> 
> Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>

We have a positive review on the API, but the implementation is still
rather rough. Is there any consensus on whether the APIs should make it
into 5.2, or do I slip the rest of this series into 5.3?  (The decision
may be made for me, if DV cuts the -rc to start the freeze...)

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]

  Powered by Linux