Gilad Chaplik has posted comments on this change. Change subject: core: Use pin to host only when present in cluster policy ......................................................................
Patch Set 4: (3 comments) .................................................... File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java Line 184 Line 185 Line 186 Line 187 Line 188 I agree with removal of checkDestinationHost, but if I recall Doron objected to it. Line 187 Line 188 Line 189 Line 190 Line 191 basically we just need to remove this if condition to solve the bug. optimization can be added (if(vdsList.size() == 1) return null) Line 241 Line 242 Line 243 Line 244 Line 245 same. -- To view, visit http://gerrit.ovirt.org/19784 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If2c0496327c9f692b0dbe92e1cf481f900f4a747 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
