Change in ovirt-engine[ovirt-engine-3.3]: core: Use pin to host only when present in cluster policy

iheim at redhat.com iheim at redhat.com
Tue Oct 15 17:14:08 UTC 2013


Itamar Heim has submitted this change and it was merged.

Change subject: core: Use pin to host only when present in cluster policy
......................................................................


core: Use pin to host only when present in cluster policy

This removes the explicit pin to host check in scheduling.
The check is already present in cluster policy and will be used
when the policy unit is selected by the administrator.

It also restores the optimization we had in the code that
used Pin To Host logic as the first filter for the provided
cluster policies.

Change-Id: If2c0496327c9f692b0dbe92e1cf481f900f4a747
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1007282
Signed-off-by: Martin Sivak <msivak at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
A packaging/dbscripts/upgrade/03_03_0990_pin_to_host_first_in_powersaving_policy.sql
3 files changed, 14 insertions(+), 54 deletions(-)

Approvals:
  Itamar Heim: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If2c0496327c9f692b0dbe92e1cf481f900f4a747
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list