proxmox-mirrors/pve-manager
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-manager synced 2025-08-03 11:36:47 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

ui: guest migrate: allow mapped devices for offline migrations

Browse Source 
if the migration is an offline migration and when the mapping on the
target node exists, otherwise not

this does not change the behaviour for 'raw' devices in the config
those can still be forced to be migrated, like before

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2023-06-16 15:05:40 +02:00 committed by Thomas Lamprecht
parent 70eb18e8f0
commit 02f14aa838
1 changed files with 42 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
www/manager6/window/Migrate.js
View File

@ -219,36 +219,68 @@ Ext.define('PVE.window.Migrate', {
let target = me.lookup('pveNodeSelector').value;
if (target.length && !migrateStats.allowed_nodes.includes(target)) {
let disallowed = migrateStats.not_allowed_nodes[target];
let missingStorages = disallowed.unavailable_storages.join(', ');
if (disallowed.unavailable_storages !== undefined) {
let missingStorages = disallowed.unavailable_storages.join(', ');
migration.possible = false;
migration.preconditions.push({
text: 'Storage (' + missingStorages + ') not available on selected target. ' +
'Start VM to use live storage migration or select other target node',
severity: 'error',
});
migration.possible = false;
migration.preconditions.push({
text: 'Storage (' + missingStorages + ') not available on selected target. ' +
'Start VM to use live storage migration or select other target node',
severity: 'error',
});
}
if (disallowed['unavailable-resources'] !== undefined) {
let unavailableResources = disallowed['unavailable-resources'].join(', ');
migration.possible = false;
migration.preconditions.push({
text: 'Mapped Resources (' + unavailableResources + ') not available on selected target. ',
severity: 'error',
});
}
}
}
if (migrateStats.local_resources.length) {
let blockingResources = [];
let mappedResources = migrateStats['mapped-resources'] ?? [];
for (const res of migrateStats.local_resources) {
if (mappedResources.indexOf(res) === -1) {
blockingResources.push(res);
}
}
if (blockingResources.length) {
migration.hasLocalResources = true;
if (!migration.overwriteLocalResourceCheck || vm.get('running')) {
migration.possible = false;
migration.preconditions.push({
text: Ext.String.format('Can\'t migrate VM with local resources: {0}',
migrateStats.local_resources.join(', ')),
blockingResources.join(', ')),
severity: 'error',
});
} else {
migration.preconditions.push({
text: Ext.String.format('Migrate VM with local resources: {0}. ' +
'This might fail if resources aren\'t available on the target node.',
migrateStats.local_resources.join(', ')),
blockingResources.join(', ')),
severity: 'warning',
});
}
}
if (mappedResources && mappedResources.length) {
if (vm.get('running')) {
migration.possible = false;
migration.preconditions.push({
text: Ext.String.format('Can\'t migrate running VM with mapped resources: {0}',
mappedResources.join(', ')),
severity: 'error',
});
}
}
if (migrateStats.local_disks.length) {
migrateStats.local_disks.forEach(function(disk) {
if (disk.cdrom && disk.cdrom === 1) {