[Kimchi-devel] [PATCH] <Fix problems of edit a guest containssnapshots>

[Crístian Viana]

On 07-01-2015 23:38, Zongmei Gou wrote:
> 1. snap.listAllChildren() in _backup_snapshots.py need flag in older 
> libvirt versions, as I used "0.10.2", it raised "listAllChildren() 
> takes exactly 2 arguments (1 given)"
>
Great! I'll fix that.

> 2. UI transfer param 'name' no matter you update vm name or not (of 
> course while you edit a vm), so do we only need to bakup and restore 
> the snapshots while vm name is really updated ?

Well, IMO that's a UI bug. The UI shouldn't send the parameter 'name' 
when the VM name isn't changed. I think we should keep our patches like 
this (i.e. updating only the fields explicitly passed) and the UI should 
be updated later so it works better under the hood.

I'll send the patch with the fix (1) above.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/kimchi-devel/attachments/20150113/27cc27a9/attachment.html>