Doron Fediuck has posted comments on this change. Change subject: [WIP] MOM Integration ......................................................................
Patch Set 6: (1 inline comment) .................................................... File vdsm/clientIF.py Line 92: AFAIR, MOM is Memory Overcommitment Manager, and the policy rules will come from ovirt-engine. My initial comment was from the dev's point of view, who (after this change) will need to check whether ksm or mom is in use. Going forward a single entity in vdsm side will make it simpler, and we can later decide if it represents only MOM or something more generic. If you want to continue discussing it, I suggest move it to some mailing list, although I already said we can leave this for now and revisit in next generation. -- To view, visit http://gerrit.ovirt.org/2367 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I61e68f72e9115a913d5bc0f4903b906b0d0cce2f Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Mark Wu <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: Shu Ming <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
