[pve-devel] [PATCH manager v2 4/6] ui ProcessorEdit: remove total core count displayfield

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Oct 5 15:17:39 CEST 2017


Make space for merging CPUOptions into ProcessorEdit.

Instead, make it clearer that the 'cores' option is really a 'cores
per socket' option

Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---

no changes

 www/manager6/qemu/ProcessorEdit.js | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/www/manager6/qemu/ProcessorEdit.js b/www/manager6/qemu/ProcessorEdit.js
index 1ac6db26..627d0c93 100644
--- a/www/manager6/qemu/ProcessorEdit.js
+++ b/www/manager6/qemu/ProcessorEdit.js
@@ -16,7 +16,6 @@ Ext.define('PVE.qemu.ProcessorInputPanel', {
 	setTotalCores: function() {
 	    var sockets = this.lookup('sockets').getValue();
 	    var cores = this.lookup('cores').getValue();
-	    this.lookup('totalcores').setValue(sockets * cores);
 	}
     },
 
@@ -70,7 +69,7 @@ Ext.define('PVE.qemu.ProcessorInputPanel', {
 	    minValue: 1,
 	    maxValue: 128,
 	    value: '1',
-	    fieldLabel: gettext('Cores'),
+	    fieldLabel: gettext('Cores/Socket'),
 	    allowBlank: false
 	},
 	{
@@ -87,13 +86,6 @@ Ext.define('PVE.qemu.ProcessorInputPanel', {
 	    name: 'cputype',
 	    value: '__default__',
 	    fieldLabel: gettext('Type')
-	},
-	{
-	    xtype: 'displayfield',
-	    fieldLabel: gettext('Total cores'),
-	    name: 'totalcores',
-	    reference: 'totalcores',
-	    value: '1'
 	}
     ],
 
-- 
2.11.0





More information about the pve-devel mailing list