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

Code Review gerrit at ovirt.org
Mon Jan 9 16:43:07 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan 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
  Andrej Krejcir: Looks good to me, but someone else must approve
  Yanir Quinn: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I466a72a546a048a177c55ea2ca6a1a0b697bd53e
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan 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