[pve-devel] applied: [PATCH qemu-server v6 0/3] Add qm importdisk command

Fabian Grünbichler f.gruenbichler at proxmox.com
Thu Jun 1 11:40:57 CEST 2017


with a bit of cleanup in the error handling parts

On Thu, Jun 01, 2017 at 10:26:36AM +0200, Emmanuel Kasper wrote:
> V6 changes:
>  * use die if false(); instead of false or die(); in qm.pm
>  * use vmconfig_apply_pending and vmconfig_hotplug_pending to update vm conf in ImportDisk.pm
>  * make sure we display the error of the eval block in ImportDisk.pm
>  * in the do_import sub in ImportDisk.pm do not retest the path to allocated disk as this was already tested
> 
> v5 changes:
>  * QemuServer.pm
>      * rename choose_dst_disk_format to resolve_dst_disk_format
>      ( as the end user chooses, but the computer resolve)
>      * fix a uninitialized value warning when format was unspecified
>      when called without $srv_volname being set
>  * ImportDisk.pm
>      * remove data dumper
>      * use file_size_info from Storage.pm module
>      * always call choose_dst_disk_format
>      * fix comment
>      * remove $dst hash ref
>      * remove unnecessary vm_is_volid_owner()
>      * merge create_unused_ and create_active_ in my $create_drive
>      * check if requested drive_name is valid before allocating disk image
>      * check if requested drive_name is available once we locked (should not happpen but ...)
>      * extract optional params early to avoid lots of hash dereferencing later on
>  * qm.pm
>      * use the right format for storage command line parameter
>      * add extra parameters checks:
>        * vm should not be locked
>        * source file must be a regular file
>        * storage must be valid
> 
> 
> v4 changes:
>  * add optional switch -format to select between raw|qcow2|vmdk, like qm move_disk.
>  For this the code handling target format in clone_disk was factored out in a
>  separate sub that we reuse
>  * fix overwriting of vm.conf when importdisk and ovfdisk were combined
>  (was bitten by autovivification)
>  * rename command and package from diskimport to importdisk, because we
>  mostly have listsnapshots, move_disk, etc ... (verb + name)
>  * useful nitpicks from Thomas about code formatting
> 
> v3 changes:
>  * when command lines parameters are mandatory, do not hide them behind switches
> 
> Emmanuel Kasper (3):
>   Refactor the resolution of target disk format in own sub
>   Add ImportDisk module to import external disk images into a VM
>   Add new qm command 'importdisk' to import external disk images
> 
>  PVE/CLI/qm.pm                | 51 +++++++++++++++++++++++
>  PVE/QemuServer.pm            | 33 ++++++++++-----
>  PVE/QemuServer/ImportDisk.pm | 98 ++++++++++++++++++++++++++++++++++++++++++++
>  PVE/QemuServer/Makefile      |  1 +
>  4 files changed, 172 insertions(+), 11 deletions(-)
>  create mode 100755 PVE/QemuServer/ImportDisk.pm
> 
> -- 
> 2.11.0
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel




More information about the pve-devel mailing list