]> git.proxmox.com Git - pve-manager.git/commit
ui: import: increase CPU limits to better match ESXi
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Wed, 13 Mar 2024 11:54:06 +0000 (12:54 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Thu, 14 Mar 2024 14:17:43 +0000 (15:17 +0100)
commit194eb3a850be51208d0b56ebcdb36e05be14b8d0
tree0b9d949f8a527ba58bf586ca92e6f7c6d1b20d4a
parent2d0bf566efd23e263fc94bd36df376bbe7d5671d
ui: import: increase CPU limits to better match ESXi

On ESXi one can have at max 768 vCPUs and that over maximal 128
sockets. Further, by default ESXi will favor adding single-cpu
sockets, so if one just enters "give me 8 vCPUs", it will be 8
single-core sockets.

As we only supported up to 4 sockets in the web UI, which caused this
field to show up as invalid on initial load for such VMs, which is not
nice.

Having that many sockets is rather odd, so for us it doesn't make
sense to allow more than the ESXi limit that, but having more vCPUs
might make sense, so allow up to 1024 for that for now.

Note that QEMU and our API backend doesn't really cares, the kernels
NR_MAX_CPU is the limit there. Albeit it is questionable if using more
vCPU threads as there are host threads has some sense in any setup.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
www/manager6/window/GuestImport.js