[pve-devel] [PATCH qemu-server 0/2] Add qm diskimport command

[Emmanuel Kasper]

On 04/27/2017 07:56 PM, Alexandre DERUMIER wrote:
> Hi Emmanuel,
> 
> I have tested both ovf and disk import with differents ovf found on the net, 
> 
> it's working fine for me. (We just need to add import of disks in import ovf)
> 
> 
> 
>>> Combined with the OVF parser which was submitted to the list last month 
>>> ( [PATCH qemu-server v4 0/5] Initial support for OVF import ) this should
>>> allows direct import of ovf compatible exports from other hypervisors.
> 
> Just note that disk import need to be rebase to apply on top of ovf import. (Makefile and require in QemuServer.pm).

thanks for testing ! Yes a minor rebase is needed, depending on which
patch serie is applied first, I will rebase the second one.

> I just wonder if both imports could be merged ?
> 
> 
> qm import vmid -source dummy.ovf -storage targetstorage   -> import full vm conf, create a new vm and import disks
> 
> qm import vmid -source dummy.vmdk -storage targetstorage   -> import a disk in a existing vm or create a new vmid with basic config if vmid don't exist.
> 
> 
> just try to detect if file is an .ovf, if not, assume that it's a disk and use qemu-img convert

I prefer to keep the two commands separate,since in the long run I was
thinking of extending the ovfimport with ova support, where the ova is a
tar file containing disk image + ovf file + checksum/signature

Having a command which accepts three different kind of inputs would be
then too confusing IMHO.