Gilad Chaplik has posted comments on this change.

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


Patch Set 1:

(1 comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java
Line 201:                 }
Line 202:             }
Line 203:             if (vdsList != null && vdsList.contains(destHostId)) {
Line 204:                 return destHostId;
Line 205:             }
plz remove this if condition.
Line 206:             if (policy.getFunctions() == null || 
policy.getFunctions().isEmpty()) {
Line 207:                 return vdsList.get(0).getId();
Line 208:             }
Line 209:             Guid bestHost = runFunctions(policy.getFunctions(), 
vdsList, vm, parameters);


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If2c0496327c9f692b0dbe92e1cf481f900f4a747
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Martin Sivák <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to