[pve-devel] [PATCH qemu-server] api: vmlist: add plugged cpu count to vmlist table

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Jan 17 16:27:31 CET 2025


Am 17.01.25 um 16:10 schrieb Daniel Kral:
> What do you think about fixing the problem from the other side: I could 
> do a follow up where it's not possible to actively assign a VM more 
> resources than currently available, since it would not boot anyway? I 
> think there's a negligible amount of users who'd want to configure more 
> resources and only later upgrade to the required primary memory or a 
> heavier CPU (wrt to their core count).

Such a VM might not boot on it's current node, but it could boot on
another node with more CPU cores. Often not a big issue, but we avoid
such restrictions as they often create more friction without providing
that much help – a UI only hint in case of setting more cores than the
current node supports might be better here.

Maybe best to just ignore the glitch in the output for now, or indeed
duplicate the calculation, or look into why vmstatus does it that way
and see if it could be unclamped there (or a new property is warranted).

btw. you use the "api" prefix for this patch, but it should be "cli",
as this is not changing the (REST) API in any way.




More information about the pve-devel mailing list