Tal Nisan has submitted this change and it was merged.
Change subject: HE: Store maximum number of vcpus to the OVF file
......................................................................
HE: Store maximum number of vcpus to the OVF file
This patch adds the computed number of vCPUs to the OVF file for
VMs. The number can then be used by the hosted engine tooling to
request the proper topology from VDSM when starting the hosted engine
VM and that in turn should enable CPU hotplug for the VM.
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1304387
Change-Id: If48ec31ada5888dc4ed90c30db044a188481fc0c
Signed-off-by: Martin Sivak <msivak(a)redhat.com>
(cherry picked from commit 0422d54d7295ddd3d6b33a708a1d51cb1b41afc4)
---
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/disk/image/RemoveImageCommandTest.java
A
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmCpuCountHelper.java
M
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfWriter.java
M
backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfManagerTest.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilderBase.java
5 files changed, 61 insertions(+), 15 deletions(-)
Approvals:
Martin Sivák: Verified
Jenkins CI: Passed CI tests
Roy Golan: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/61307
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If48ec31ada5888dc4ed90c30db044a188481fc0c
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>