[pve-devel] [PATCH] check has_feature copy for full clone

[Dietmar Maurer]

wait, I guess I have an idea how to fix that cleanly.

> -----Original Message-----
> From: pve-devel-bounces at pve.proxmox.com [mailto:pve-devel-
> bounces at pve.proxmox.com] On Behalf Of Alexandre Derumier
> Sent: Donnerstag, 02. Mai 2013 06:09
> To: pve-devel at pve.proxmox.com
> Subject: [pve-devel] [PATCH] check has_feature copy for full clone
> 
> works with or without snapname.
> 
> Signed-off-by: Alexandre Derumier <aderumier at odiso.com>
> ---
>  PVE/API2/Qemu.pm |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index 93458b0..9baa16e
> 100644
> --- a/PVE/API2/Qemu.pm
> +++ b/PVE/API2/Qemu.pm
> @@ -1923,6 +1923,8 @@ __PACKAGE__->register_method({
> 
>  	    my $sharedvm = &$check_storage_access_copy($rpcenv, $authuser,
> $storecfg, $oldconf, $storage);
> 
> +	    die "Full copy feature is not available" if $param->{full} &&
> +!PVE::QemuServer::has_feature('copy', $oldconf, $storecfg, $snapname);
> +
>  	    die "can't copy VM to node '$target' (VM uses local storage)\n" if
> $target && !$sharedvm;
> 
>  	    my $conffile = PVE::QemuServer::config_file($newid);
> --
> 1.7.10.4
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel