Change in ovirt-engine[master]: Add scheduling filter that prevents migration to same host

gchaplik at redhat.com gchaplik at redhat.com
Thu Dec 18 08:25:54 UTC 2014


Gilad Chaplik has submitted this change and it was merged.

Change subject: Add scheduling filter that prevents migration to same host
......................................................................


Add scheduling filter that prevents migration to same host

Inhibit migration if destination hostname resolves
to source host. In that case, libvirt will abort
anyway, so this check just prevents libvirt error
from happening.

In addition, avoid migrations to a host which
resolves to localhost, which is rarely, if ever,
a good thing.

Change-Id: I42021452205f3dc070388a43de92c8fd5ed6dbf2
Bug-Url: https://bugzilla.redhat.com/1107650
Signed-off-by: Martin Sivák <msivak at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/PolicyUnitImpl.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/MigrationPolicyUnit.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
A packaging/dbscripts/upgrade/03_06_0640_add_migration_policy_unit.sql
8 files changed, 89 insertions(+), 9 deletions(-)

Approvals:
  Gilad Chaplik: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/34700
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I42021452205f3dc070388a43de92c8fd5ed6dbf2
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list