[pve-devel] applied: [PATCH v4 qemu-server 1/3] fix #1013 : migrate : sync_disk : --targetstorage with offline disk

[Thomas Lamprecht]

On 12/20/18 9:55 AM, Wolfgang Bumiller wrote:
> From: Alexandre Derumier <aderumier at odiso.com>
> 
> targetsid was not used, for disk unused (offline copy)
> 
> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> Acked-by: Wolfgang Bumiller <w.bumiller at proxmox.com>
> ---
> No changes to v3
> 
>  PVE/QemuMigrate.pm | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/PVE/QemuMigrate.pm b/PVE/QemuMigrate.pm
> index e9e9075..0bae076 100644
> --- a/PVE/QemuMigrate.pm
> +++ b/PVE/QemuMigrate.pm
> @@ -446,6 +446,7 @@ sub sync_disks {
>  
>  	foreach my $volid (keys %$local_volumes) {
>  	    my ($sid, $volname) = PVE::Storage::parse_volume_id($volid);
> +	    my $targetsid = $self->{opts}->{targetstorage} ? $self->{opts}->{targetstorage} : $sid;
>  	    if ($self->{running} && $self->{opts}->{targetstorage} && $local_volumes->{$volid}->{ref} eq 'config') {
>  		push @{$self->{online_local_volumes}}, $volid;
>  	    } else {
> @@ -453,7 +454,7 @@ sub sync_disks {
>  		push @{$self->{volumes}}, $volid;
>  		my $insecure = $self->{opts}->{migration_type} eq 'insecure';
>  		my $with_snapshots = $local_volumes->{$volid}->{snapshots};
> -		PVE::Storage::storage_migrate($self->{storecfg}, $volid, $self->{ssh_info}, $sid,
> +		PVE::Storage::storage_migrate($self->{storecfg}, $volid, $self->{ssh_info}, $targetsid,
>  					      undef, undef, undef, undef, $insecure, $with_snapshots);
>  	    }
>  	}
> 

applied all three patches, thanks!