[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