[pve-devel] [PATCH manager 3/3] Allow setting no target storage and make it default

Fabian Ebner f.ebner at proxmox.com
Thu Apr 30 12:59:06 CEST 2020


so the current disk locations can be preserved even if
there are multiple local disks. And users don't have to
manually select the current storage if there is only one
local disk.

Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
---

Not too happy about the "use current layout" text. Maybe
somebody has a better idea.

 www/manager6/window/Migrate.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/www/manager6/window/Migrate.js b/www/manager6/window/Migrate.js
index 20e057ad..8f3cf1c2 100644
--- a/www/manager6/window/Migrate.js
+++ b/www/manager6/window/Migrate.js
@@ -119,7 +119,7 @@ Ext.define('PVE.window.Migrate', {
 	    if (vm.get('migration.with-local-disks')) {
 		params['with-local-disks'] = 1;
 	    }
-	    if (vm.get('migration.with-local-disks')) {
+	    if (vm.get('migration.with-local-disks') && values.targetstorage) {
 		params.targetstorage = values.targetstorage;
 	    }
 
@@ -343,6 +343,9 @@ Ext.define('PVE.window.Migrate', {
 			    name: 'targetstorage',
 			    fieldLabel: gettext('Target storage'),
 			    storageContent: 'images',
+			    allowBlank: true,
+			    autoSelect: false,
+			    emptyText: 'use current layout',
 			    bind: {
 				hidden: '{!migration.with-local-disks}',
 			    }
-- 
2.20.1





More information about the pve-devel mailing list