Oved Ourfali has posted comments on this change. Change subject: DO NOT SUBMIT - wip - core: adding service strategy for gluster integration ......................................................................
Patch Set 4: Changes in latest patch: 1. Change name s/policy/strategy + more changes 2. Change method names + logic to be more generic 3. Still using the Factory, and not passing the strategy instance as a parameter. In the future, when the factory will be more complex, we should consider optimizing that. -- To view, visit http://gerrit.ovirt.org/2387 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5edc263dae23236015260d83a933e8a3a93e4523 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Livnat Peer <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Shireesh Anjal <[email protected]> Gerrit-Reviewer: Shireesh Anjal <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
