Martin Sivák has uploaded a new change for review. 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. Change-Id: Id5a49813d433a10debe607f17783057d7d4a767b Signed-off-by: Martin Sivak <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PinToHostPolicyUnit.java 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/86/19786/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PinToHostPolicyUnit.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PinToHostPolicyUnit.java index 6e43b78..dba2ba9 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PinToHostPolicyUnit.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PinToHostPolicyUnit.java @@ -1,5 +1,6 @@ package org.ovirt.engine.core.bll.scheduling.policyunits; +import java.util.ArrayList; import java.util.Arrays; import java.util.List; import java.util.Map; @@ -28,6 +29,7 @@ // if flow reaches here, the VM is pinned but there is no dedicated host. messages.add(VdcBllMessages.ACTION_TYPE_FAILED_VM_IS_PINNED_TO_HOST.toString()); + return new ArrayList<>(); } return hosts; -- To view, visit http://gerrit.ovirt.org/19786 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id5a49813d433a10debe607f17783057d7d4a767b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
