From Arik Hadas <ahadas(a)redhat.com>:
Arik Hadas has submitted this change and it was merged.
Change subject: core: Adjust the limit on max VCPUs
......................................................................
core: Adjust the limit on max VCPUs
The maximum number of VCPUs must be multiplication of sockets * cores. This is
ensured by the formula computing the maximum number of VCPUs, but the
configuration value may not satisfy that constraint. So if we are limited by
the configuration value then we must check and adjust that value.
This is mostly theoretical in 4.1+ since the current configuration value (288)
is higher than the APIC limit (255) so it should cause no trouble in the
default 4.1 configuration.
Change-Id: I857e1a85950a863752e02a30066f682fff04a7c7
Signed-off-by: Milan Zamazal <mzamazal(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmCpuCountHelper.java
1 file changed, 9 insertions(+), 3 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Arik Hadas: Looks good to me, approved
Milan Zamazal: Verified
--
To view, visit
https://gerrit.ovirt.org/72378
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I857e1a85950a863752e02a30066f682fff04a7c7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzamazal(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>