Change in vdsm[master]: [WIP] MOM Integration

dfediuck at redhat.com dfediuck at redhat.com
Wed May 2 08:56:52 UTC 2012


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 <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Shu Ming <shuming at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list