[pve-devel] [PATCH 1/5] add qemu_img_convert

[Alexandre DERUMIER]

>>I think create_vm is already too complex to add more functionality.
>>What do you think? 

Yes, I also was unhappy with extending qm create

>>My plan was to get this basic functionality stable so
>>that we can include this with 3.0?

Yes, I think it could be a start, we can add features step by step.
When do you plan to release 3.0 ?


I'll test your code today




----- Mail original ----- 

De: "Dietmar Maurer" <dietmar at proxmox.com> 
À: "Alexandre Derumier" <aderumier at odiso.com>, pve-devel at pve.proxmox.com 
Envoyé: Lundi 29 Avril 2013 11:03:50 
Objet: RE: [pve-devel] [PATCH 1/5] add qemu_img_convert 

I finally applied this one and a first version of copy_vm. 

I moved the code into an extra API call, because copy/clone has 
different locking requirements than create_vm. We now create 
a shared lock on the source VM. Also, permission checks are slightly 
different, and I think create_vm is already too complex to add more functionality. 

What do you think? My plan was to get this basic functionality stable so 
that we can include this with 3.0? 


> -----Original Message----- 
> From: pve-devel-bounces at pve.proxmox.com [mailto:pve-devel- 
> bounces at pve.proxmox.com] On Behalf Of Alexandre Derumier 
> Sent: Montag, 29. April 2013 08:41 
> To: pve-devel at pve.proxmox.com 
> Subject: [pve-devel] [PATCH 1/5] add qemu_img_convert 
> 
> also work with snapshot as source for qcow2,rbd,sheepdog. 
>