Change in ovirt-engine[ovirt-engine-3.3]: core: Fix PinToHostPolicyUnit when PinToHost is enabled

iheim at redhat.com iheim at redhat.com
Tue Oct 15 17:13:50 UTC 2013


Itamar Heim has submitted this change and it was merged.

Change subject: core: Fix PinToHostPolicyUnit when PinToHost is enabled
......................................................................


core: Fix PinToHostPolicyUnit when PinToHost is enabled

We have to return an empty list of available hosts when PinToHost
is enabled and the pinned Host was already filtered out.

Otherwise the whole PolicyUnit has no effect.

This is fixing one issue out of two that caused the
referenced bug. The issues are independent on each other
and so I did want to preserve the history.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1007282

Change-Id: Id5a49813d433a10debe607f17783057d7d4a767b
Signed-off-by: Martin Sivak <msivak at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PinToHostPolicyUnit.java
1 file changed, 6 insertions(+), 0 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id5a49813d433a10debe607f17783057d7d4a767b
Gerrit-PatchSet: 5
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