Re: [libvirt] [RFC]: Snapshot API

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

 



On 03/23/2010 01:23 PM, Chris Lalancette wrote:
>>> } virDomainSnapshotDeactivate;
>>> >> int virDomainSnapshotDeactivate(virDomainSnapshotPtr snapshot,
>>> >>                                unsigned int flags);
>> > 
>> > I'm not sure if virDomainSnapshotDeactivate is a good name.
> I agree it's not a great name.  I didn't like Dan's original
> proposal of "virDomainSnapshotDelete", though, since it doesn't
> exactly seem to fit the situation either.  Any more suggestions for
> a name?

I agree that Delete is misleading, but it kind of works (as long as you
remember that in the merge case, the data being deleted is not the
snapshot parameter that you passed to the function call, but the delta
in state between the more-recent content that is being rolled back to
the state it was earlier during the snapshot).  Maybe some other ideas
would work: virDomainSnapshotCleanup or virDomainSnapshotRecycle?  After
all, the goal of this API is to reduce the amount of storage pool being
used to store snapshot images.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
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

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