public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH manager v4] ui: processor: edit: remove UI limit on socket count
@ 2026-07-10  9:52 Maximiliano Sandoval
  2026-07-10 11:59 ` applied: " Dominik Csapak
  0 siblings, 1 reply; 2+ messages in thread
From: Maximiliano Sandoval @ 2026-07-10  9:52 UTC (permalink / raw)
  To: pve-devel

The backend imposes no limit on the number of sockets, so the UI should not
either. Remove the arbitrary cap and let users set socket counts as high as the
backend allows.

Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
---

Notes:
    Differences from v3:
    - Change commit title and description
    
    Differences from v2:
    - Remove the maxValue setting
    
    Differences from v1:
    - Set the maxValue to 8

 www/manager6/qemu/ProcessorEdit.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/www/manager6/qemu/ProcessorEdit.js b/www/manager6/qemu/ProcessorEdit.js
index 719f771a7..f640de421 100644
--- a/www/manager6/qemu/ProcessorEdit.js
+++ b/www/manager6/qemu/ProcessorEdit.js
@@ -129,7 +129,6 @@ Ext.define('PVE.qemu.ProcessorInputPanel', {
             xtype: 'proxmoxintegerfield',
             name: 'sockets',
             minValue: 1,
-            maxValue: 4,
             value: '1',
             fieldLabel: gettext('Sockets'),
             allowBlank: false,
-- 
2.47.3





^ permalink raw reply related	[flat|nested] 2+ messages in thread

* applied: [PATCH manager v4] ui: processor: edit: remove UI limit on socket count
  2026-07-10  9:52 [PATCH manager v4] ui: processor: edit: remove UI limit on socket count Maximiliano Sandoval
@ 2026-07-10 11:59 ` Dominik Csapak
  0 siblings, 0 replies; 2+ messages in thread
From: Dominik Csapak @ 2026-07-10 11:59 UTC (permalink / raw)
  To: pve-devel, Maximiliano Sandoval

On Fri, 10 Jul 2026 11:52:15 +0200, Maximiliano Sandoval wrote:
> The backend imposes no limit on the number of sockets, so the UI should not
> either. Remove the arbitrary cap and let users set socket counts as high as the
> backend allows.

Applied, thanks!

[1/1] ui: processor: edit: remove UI limit on socket count
      commit: b0b650c16c520eaaf62db54810dcf8d66bc4249f




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-07-10 11:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-10  9:52 [PATCH manager v4] ui: processor: edit: remove UI limit on socket count Maximiliano Sandoval
2026-07-10 11:59 ` applied: " Dominik Csapak

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal