From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id A7149E14D for ; Thu, 21 Apr 2022 13:27:37 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 614C71E336 for ; Thu, 21 Apr 2022 13:27:13 +0200 (CEST) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [94.136.29.106]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS id D1AF21E272 for ; Thu, 21 Apr 2022 13:27:07 +0200 (CEST) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id ABCCB4265A for ; Thu, 21 Apr 2022 13:27:07 +0200 (CEST) From: Fabian Ebner To: pve-devel@lists.proxmox.com Date: Thu, 21 Apr 2022 13:26:59 +0200 Message-Id: <20220421112659.74011-14-f.ebner@proxmox.com> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20220421112659.74011-1-f.ebner@proxmox.com> References: <20220421112659.74011-1-f.ebner@proxmox.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 0 AWL 0.088 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: [pve-devel] [PATCH v2 manager 3/3] ui: restore: allow preserving disks X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Apr 2022 11:27:37 -0000 Signed-off-by: Fabian Ebner --- New in v2. Dependency bump for proxmox-widget-toolkit needed www/manager6/Makefile | 1 + www/manager6/grid/RestoreDiskGrid.js | 125 +++++++++++++++++++++++++++ www/manager6/window/Restore.js | 35 +++++++- 3 files changed, 160 insertions(+), 1 deletion(-) create mode 100644 www/manager6/grid/RestoreDiskGrid.js diff --git a/www/manager6/Makefile b/www/manager6/Makefile index 2c7b1e70..93f9f9c6 100644 --- a/www/manager6/Makefile +++ b/www/manager6/Makefile @@ -80,6 +80,7 @@ JSSRC= \ grid/PoolMembers.js \ grid/Replication.js \ grid/ResourceGrid.js \ + grid/RestoreDiskGrid.js \ panel/ConfigPanel.js \ panel/BackupJobPrune.js \ panel/HealthWidget.js \ diff --git a/www/manager6/grid/RestoreDiskGrid.js b/www/manager6/grid/RestoreDiskGrid.js new file mode 100644 index 00000000..5384a5e6 --- /dev/null +++ b/www/manager6/grid/RestoreDiskGrid.js @@ -0,0 +1,125 @@ +Ext.define('pve-restore-disk', { + extend: 'Ext.data.Model', + fields: [ + { name: 'preserve', type: 'bool' }, + { name: 'drivename', type: 'string' }, + { name: 'vm', type: 'string' }, + { name: 'backup', type: 'string' }, + ], +}); + +Ext.define('PVE.grid.RestoreDiskGrid', { + extend: 'Ext.grid.Panel', + alias: ['widget.pveRestoreDiskGrid'], + + vmConfig: undefined, + backupConfig: undefined, + + setVMConfig: function(config) { + let me = this; + + if (me.vmConfig) { + throw "vm config already set"; + } + me.vmConfig = config; + + if (me.vmConfig && me.backupConfig) { + me.updateData(); + } + }, + + setBackupConfig: function(config) { + let me = this; + + if (me.backupConfig) { + throw "backup config already set"; + } + me.backupConfig = config; + + if (me.vmConfig && me.backupConfig) { + me.updateData(); + } + }, + + updateData: function() { + let me = this; + + let data = []; + + Object.keys(me.vmConfig).concat(Object.keys(me.backupConfig)).forEach(function(key) { + if (!key.match(PVE.Utils.bus_match) && !key.match(/^(efidisk|tpmstate)\d+$/)) { + return; + } + + if (data.find(item => item.drivename === key)) { + return; + } + + data.push({ + drivename: key, + preserve: false, + vm: me.vmConfig[key], + backup: me.backupConfig[key], + }); + }); + + me.getStore().setData(data); + }, + + preserveDrives: function() { + let drives = []; + + this.getStore().getData().each(function(item) { + if (item.data.preserve) { + drives.push(item.data.drivename); + } + }); + + return drives; + }, + + + store: { + model: 'pve-restore-disk', + sorters: 'drivename', + }, + + columns: [ + { + xtype: 'checkcolumn', + header: gettext('Preserve'), + dataIndex: 'preserve', + width: 80, + }, + { + header: gettext('Drive'), + dataIndex: 'drivename', + width: 80, + }, + { + header: gettext('Current'), + dataIndex: 'vm', + flex: 1, + renderer: function(drive, metaData, record) { + if (record.data.preserve) { + metaData.tdCls = 'proxmox-good-row'; + } else if (!record.data.backup) { + metaData.tdCls = 'proxmox-good-row'; + return `(${gettext('as unused')}) ${drive}`; + } + return drive; + }, + }, + { + header: gettext('Backup'), + dataIndex: 'backup', + flex: 1, + renderer: function(drive, metaData, record) { + if (!record.data.preserve && drive) { + metaData.tdCls = 'proxmox-good-row'; + } + return drive; + }, + }, + ], +}); diff --git a/www/manager6/window/Restore.js b/www/manager6/window/Restore.js index 23c244f3..8b0134b8 100644 --- a/www/manager6/window/Restore.js +++ b/www/manager6/window/Restore.js @@ -18,6 +18,7 @@ Ext.define('PVE.window.Restore', { afterRender: function() { let view = this.getView(); + let diskGrid = view.lookupReference('restoreDiskGrid'); Proxmox.Utils.API2Request({ url: `/nodes/${view.nodename}/vzdump/extractconfig`, @@ -29,6 +30,7 @@ Ext.define('PVE.window.Restore', { failure: (response) => Ext.Msg.alert('Error', response.htmlStatus), success: function(response, options) { let storagesAvailable = true; + let config = {}; response.result.data.split('\n').forEach(function(line) { let match = line.match( @@ -43,6 +45,8 @@ Ext.define('PVE.window.Restore', { match = line.match(/^([^:]+):\s*(\S+)\s*$/); if (match) { let [_, key, value] = match; + config[key] = value; + switch (key) { case 'name': case 'hostname': @@ -69,8 +73,22 @@ Ext.define('PVE.window.Restore', { storagesel.allowBlank = false; storagesel.setEmptyText(''); } + + diskGrid?.setBackupConfig(config); }, }); + + if (!diskGrid) { + return; + } + + Proxmox.Utils.API2Request({ + url: `/nodes/${view.nodename}/qemu/${view.vmid}/config`, + waitMsgTarget: diskGrid, + method: 'GET', + failure: (response) => Ext.Msg.alert(gettext('Error'), response.htmlStatus), + success: (response) => diskGrid.setVMConfig(response.result.data), + }); }, }, @@ -271,6 +289,13 @@ Ext.define('PVE.window.Restore', { maxValue: 4, allowBlank: true, }); + + if (me.vmid) { + items.push({ + xtype: 'pveRestoreDiskGrid', + reference: 'restoreDiskGrid', + }); + } } me.formPanel = Ext.create('Ext.form.Panel', { @@ -331,6 +356,14 @@ Ext.define('PVE.window.Restore', { delete params.name; } + let diskGrid = me.lookupReference('restoreDiskGrid'); + if (diskGrid) { + let preserveDrives = diskGrid.preserveDrives(); + if (preserveDrives.length > 0) { + params['preserve-drives'] = preserveDrives.join(','); + } + } + var url; var msg; if (me.vmtype === 'lxc') { @@ -376,7 +409,7 @@ Ext.define('PVE.window.Restore', { Ext.apply(me, { title: title, - width: 500, + width: 700, modal: true, layout: 'auto', border: false, -- 2.30.2