Change in ovirt-engine[master]: bll: Prevent migration to the same host when balancing

Code Review gerrit at ovirt.org
Mon Jan 9 10:26:17 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

Martin Sivák has submitted this change and it was merged.

Change subject: bll: Prevent migration to the same host when balancing
......................................................................


bll: Prevent migration to the same host when balancing

This adds a new command - BalanceVm - that is the same as
MigrateVm in all aspects except one:

It allows the scheduler to select the host the VM is
currently running at and does nothing when it is selected
as the best host. All other results cause an ordinary
migration to be started.

Bug-Url: https://bugzilla.redhat.com/1410040
Change-Id: I466a72a546a048a177c55ea2ca6a1a0b697bd53e
Signed-off-by: Martin Sivak <msivak at redhat.com>
---
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BalanceVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManager.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/BasicMigrationHandler.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties
6 files changed, 47 insertions(+), 4 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Arik Hadas: Looks good to me, but someone else must approve
  Yanir Quinn: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I466a72a546a048a177c55ea2ca6a1a0b697bd53e
Gerrit-PatchSet: 15
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list