Royce Lv has posted comments on this change.
Change subject: add verb for vdsm to tuneCpu params
......................................................................
Patch Set 6: (1 inline comment)
....................................................
File vdsm/libvirtvm.py
Line 1894: def tuneCpu(self, tuneParams):
Line 1895: try:
Line 1896: self._dom.setSchedulerParameters(tuneParams)
Line 1897: for param in tuneParams:
Line 1898: self.conf[param] = tuneParams[param]
Sorry for noise, when libvirt recieve empty parameter in "setSchedulerParameters
" it will raise LookupError.
Line 1899: self.saveState()
Line 1900: return {'status': doneCode}
Line 1901: except libvirt.libvirtError, e:
Line 1902: self.log.error("TuneCpu failed:%s",
--
To view, visit
http://gerrit.ovirt.org/7492
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I09e02e09ef06ad6de45be75c3f2f913a3025750f
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Royce Lv <lvroyce(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Adam Litke <agl(a)us.ibm.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Royce Lv <lvroyce(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Ryan Harper <ryanh(a)us.ibm.com>
Gerrit-Reviewer: ShaoHe Feng <shaohef(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Zhu Bo <bozhu(a)linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server