Change in ovirt-engine[ovirt-engine-4.0]: HE: Store maximum number of vcpus to the OVF file

tnisan at redhat.com tnisan at redhat.com
Sun Jul 31 13:12:12 UTC 2016


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 at 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 at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list