they can only be migrated to nodes where there exists a mapping and if the migration is done offline
Signed-off-by: Dominik Csapak <d.csa...@proxmox.com> --- PVE/QemuMigrate.pm | 23 ++++++++++++++++++++--- 1 file changed, 20 insertions(+), 3 deletions(-) diff --git a/PVE/QemuMigrate.pm b/PVE/QemuMigrate.pm index 09cc1d83..03847c5a 100644 --- a/PVE/QemuMigrate.pm +++ b/PVE/QemuMigrate.pm @@ -210,15 +210,32 @@ sub prepare { $self->{vm_was_paused} = 1 if PVE::QemuServer::vm_is_paused($vmid); } - my $loc_res = PVE::QemuServer::check_local_resources($conf, 1); - if (scalar @$loc_res) { + my ($loc_res, $mapped_res, $not_allowed_nodes) = PVE::QemuServer::check_local_resources($conf, 1); + my $blocking_resources = []; + for my $res ($loc_res->@*) { + if (!grep($res, $mapped_res->@*)) { + push $blocking_resources->@*, $res; + } + } + if (scalar($blocking_resources->@*)) { if ($self->{running} || !$self->{opts}->{force}) { - die "can't migrate VM which uses local devices: " . join(", ", @$loc_res) . "\n"; + die "can't migrate VM which uses local devices: " . join(", ", $blocking_resources->@*) . "\n"; } else { $self->log('info', "migrating VM which uses local devices"); } } + if (scalar($mapped_res->@*)) { + my $not_available = $not_allowed_nodes->{$self->{node}}; + if ($running) { + die "can't migrate running VM which uses mapped devices: " . join(", ", $mapped_res->@*) . "\n"; + } elsif (scalar($not_available->@*)) { + die "can't migrate to '$self->{node}': missing mapped devices " . join(", ", $not_available->@*) . "\n"; + } else { + $self->log('info', "migrating VM which uses mapped local devices"); + } + } + my $vollist = PVE::QemuServer::get_vm_volumes($conf); my $storages = {}; -- 2.30.2 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel