add with-local-disks option for live storage migration

As Fabian as required,
add an extra flag "with-local-disks"  to enable live storage migration with localdisk.

default target storage is same sid than source, this can be overrided with
"targetstorage" option.

I will try improve this later, with optionnal mapping, disk by disk.

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
This commit is contained in:
Alexandre Derumier 2017-01-06 10:15:07 +01:00 committed by Dietmar Maurer
parent bd2d5fe6ff
commit 56af714629
2 changed files with 10 additions and 3 deletions

View File

@ -2723,9 +2723,15 @@ __PACKAGE__->register_method({
description => "CIDR of the (sub) network that is used for migration.",
optional => 1,
},
targetstorage => get_standard_option('pve-storage-id', {
description => "Target storage.",
"with-local-disks" => {
type => 'boolean',
description => "Enable live storage migration for local disk",
optional => 1,
},
targetstorage => get_standard_option('pve-storage-id', {
description => "Default target storage.",
optional => 1,
completion => \&PVE::QemuServer::complete_storage,
}),
},
},

View File

@ -348,6 +348,7 @@ sub sync_disks {
if ($local_volumes->{$vol} eq 'storage') {
$self->log('info', "found local disk '$vol' (via storage)\n");
} elsif ($local_volumes->{$vol} eq 'config') {
die "can't live migrate attached local disks without with-local-disks option\n" if $self->{running} && !$self->{opts}->{"with-local-disks"};
$self->log('info', "found local disk '$vol' (in current VM config)\n");
} elsif ($local_volumes->{$vol} eq 'snapshot') {
$self->log('info', "found local disk '$vol' (referenced by snapshot(s))\n");