Change in vdsm[master]: Adding utility methods and conf for CPU limit MOM integration

kobi at redhat.com kobi at redhat.com
Thu May 29 09:45:03 UTC 2014


Kobi Ianko has posted comments on this change.

Change subject: Adding utility methods and conf for CPU limit MOM integration
......................................................................


Patch Set 27:

(1 comment)

http://gerrit.ovirt.org/#/c/27258/27//COMMIT_MSG
Commit Message:

Line 3: AuthorDate: 2014-04-30 16:00:37 +0300
Line 4: Commit:     Kobi Ianko <kianku at redhat.com>
Line 5: CommitDate: 2014-05-28 18:51:09 +0300
Line 6: 
Line 7: Adding utility methods and conf for CPU limit MOM integration
> @VM.setCpuTuneQuota, or anything else that is exposed in the schema, is not
mainly because it's easier for me to think in small blocks, since I have no previous knowledge of VDSM or Python.
once there will be more tunable we can make a generic one.
for now my thoughts are that it will just delay the patch some more, performance here is not an issue (it's user click oriented)...

once we decide on this one, I'll update the comment
Line 8: 
Line 9: Adding a MOM policy to monitor CPU tuning parameter.
Line 10: Adding utility methods to vm.py to integrate with MOM's collectors and controllers
Line 11: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic502d9a4a976cd76bb6042bbb51f6cd281199631
Gerrit-PatchSet: 27
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Kobi Ianko <kobi at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Kobi Ianko <kobi at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list