Re: [PATCHv5 7/7] qemu: snapshot: Add support for external active snapshots on gluster

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

 



On 02/12/14 20:46, Eric Blake wrote:
> On 02/11/2014 09:26 AM, Peter Krempa wrote:
>> Add support for gluster backed images as sources for snapshots in the
>> qemu driver. This will also simplify adding further network backed
>> volumes as sources for snapshot in case qemu will support them.
>> ---
>>
> 
> Looks a bit confusing the number of variables we have to track as we
> swap between original and new, then fallback or commit.  But I didn't
> spot anything obviously wrong in how you did it.  I suspect that if we
> ever convert over to having domain XML track full backing chain
> semantics, we will have a lot to rewrite in this area of code to take
> advantage of that, but the future shouldn't hold up incremental progress
> in the present.  ACK.
> 

Tracking all the variables for a storage volume is really unpleasant.
I'm planing on splitting out the source part for a storage volume
(possibly including the backing chain) into a separate structure. This
will allow then for easier copying and manipulation for all the tracked
data. Unfortunately a change like that is really invasive.

Peter



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]