From 685b52f58cbbf55e510d8d0e726477dfa656ec1c Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Wed, 19 May 2021 16:51:16 +0200 Subject: [PATCH] ui: window/Migrate: avoid triggering another info request if already in-progress use the node as key, so that a target switch would still trigger a new one - else there's a slight chance that a user could get the check out-of-sync (from another node). Signed-off-by: Thomas Lamprecht --- www/manager6/window/Migrate.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/www/manager6/window/Migrate.js b/www/manager6/window/Migrate.js index 6f4759de..5122e25f 100644 --- a/www/manager6/window/Migrate.js +++ b/www/manager6/window/Migrate.js @@ -98,7 +98,6 @@ Ext.define('PVE.window.Migrate', { me.lookup('proxmoxHelpButton').setHelpConfig({ onlineHelp: vm.get(view.vmtype).onlineHelp, }); - me.checkMigratePreconditions(); me.lookup('formPanel').isValid(); }, @@ -190,11 +189,16 @@ Ext.define('PVE.window.Migrate', { } try { + if (me.fetchingNodeMigrateInfo && me.fetchingNodeMigrateInfo === vm.get('nodename')) { + return; + } + me.fetchingNodeMigrateInfo = vm.get('nodename'); let { result } = await Proxmox.Async.api2({ url: `/nodes/${vm.get('nodename')}/${vm.get('vmtype')}/${vm.get('vmid')}/migrate`, method: 'GET', }); migrateStats = result.data; + me.fetchingNodeMigrateInfo = false; } catch (error) { Ext.Msg.alert(gettext('Error'), error.htmlStatus); return;