Change in ovirt-engine[master]: broker: Do not autogenerate numatune node pinnings

rgolan at redhat.com rgolan at redhat.com
Thu Feb 11 19:14:30 UTC 2016


Roy Golan has submitted this change and it was merged.

Change subject: broker: Do not autogenerate numatune node pinnings
......................................................................


broker: Do not autogenerate numatune node pinnings

Only pin virtual NUMA nodes to the memory of specific host NUMA nodes
when it is explicitly requested. Otherwise VM NUMA nodes are implicitliy
pinned to specific host NUMA nodes (with a specific NUMA node
architecture). Then the VM can't be migrated when the host architecture
of the target host differs, although no specific NUMA configuration was
requested by the user.

Change-Id: If0d737618a692833d0162ac09303864d651b3ead
Bug-Url: https://bugzilla.redhat.com/1303842
Signed-off-by: Roman Mohr <rmohr at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/NumaSettingFactory.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java
M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/vdsbroker/NumaSettingFactoryTest.java
3 files changed, 21 insertions(+), 17 deletions(-)

Approvals:
  Roman Mohr: Verified
  Jenkins CI: Passed CI tests
  Roy Golan: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/53278
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If0d737618a692833d0162ac09303864d651b3ead
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list