Change in vdsm[master]: Utility methods and conf for blkio limits MOM integration

gchaplik at redhat.com gchaplik at redhat.com
Wed Jun 11 05:59:29 UTC 2014


Gilad Chaplik has posted comments on this change.

Change subject: Utility methods and conf for blkio limits MOM integration
......................................................................


Patch Set 1:

(5 comments)

http://gerrit.ovirt.org/#/c/28576/1/lib/vdsm/config.py.in
File lib/vdsm/config.py.in:

Line 173:         ('vm_sample_cpu_tune_interval', '15', None),
Line 174: 
Line 175:         ('vm_sample_cpu_tune_window', '2', None),
Line 176: 
Line 177:         ('vm_sample_blkio_tune_interval', '15', None),
> Please add help message explaining this value and specifying the default va
where?
Line 178: 
Line 179:         ('vm_sample_blkio_tune_window', '2', None),
Line 180: 
Line 181:         ('trust_store_path', '@TRUSTSTORE@',


Line 175:         ('vm_sample_cpu_tune_window', '2', None),
Line 176: 
Line 177:         ('vm_sample_blkio_tune_interval', '15', None),
Line 178: 
Line 179:         ('vm_sample_blkio_tune_window', '2', None),
> Please add help message explaining this value and specifying the default va
^
Line 180: 
Line 181:         ('trust_store_path', '@TRUSTSTORE@',
Line 182:             'Where the certificates and keys are situated.'),
Line 183: 


http://gerrit.ovirt.org/#/c/28576/1/tests/functional/momTests.py
File tests/functional/momTests.py:

Line 140:         for stats in candidateStats:
Line 141:             status, msg = self.s.setDiskBlkioTuneMap(
Line 142:                 stats['vmId'],
Line 143:                 disk,
Line 144:                 #TODO#)
> TODO?
TODO = to do :-)

this is a draft patch, 
can't mark it that way because gerrit doesn't support marking specific patches as drafts within a patch-set. sorry for not mentioning this earlier.
Line 145:             self.assertEqual(status, SUCCESS, msg)
Line 146: 
Line 147:     def _setPolicy(self, policy):
Line 148:         curpath = os.path.dirname(__file__)


http://gerrit.ovirt.org/#/c/28576/1/vdsm/mom.d/05-blkiotune.policy
File vdsm/mom.d/05-blkiotune.policy:

Line 1: ### Auto-blkioTune ###############################################################
Line 2: #
Line 3: #TODO
> Do you expect this to be merged?
wishful thinking :-)


http://gerrit.ovirt.org/#/c/28576/1/vdsm/virt/vm.py
File vdsm/virt/vm.py:

Line 288:         return infos
Line 289: 
Line 290:     def _sampleBlkIoTune(self):
Line 291:         infos = self._dom.schedulerParameters()
Line 292:         #TODO _getDiskBlkioTuneMap(self, disk, stats)
> Really?
LOL :-)
Line 293:         return infos
Line 294: 
Line 295:     def _diff(self, prev, curr, val):
Line 296:         return prev[val] - curr[val]


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I43be5627e5365fee1145a75e91fb5b7714e46aaf
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer 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