Noam Slomianko has uploaded a new change for review. Change subject: engine: select a host even if scheduling scoring failed ......................................................................
engine: select a host even if scheduling scoring failed If no internal scoring function is selected it can result In no host selected at all, even if some hosts did pass the filtering stage. Select the first host if none has returned from the scoring functions. Change-Id: I96082ccdfd8759619e98bc437fdb468b3b583992 Bug-Url: https://bugzilla.redhat.com/1001986 Signed-off-by: Noam Slomianko <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/29/20429/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java index eaf3737..bc91265 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java @@ -204,6 +204,9 @@ return vdsList.get(0).getId(); } Guid bestHost = runFunctions(policy.getFunctions(), vdsList, vm, parameters); + if (bestHost == null && vdsList.size() > 0) { + bestHost = vdsList.get(0).getId(); + } if (bestHost != null) { getVdsDynamicDao().updatePartialVdsDynamicCalc( bestHost, -- To view, visit http://gerrit.ovirt.org/20429 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I96082ccdfd8759619e98bc437fdb468b3b583992 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Noam Slomianko <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
