all lists on 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal