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