Adam Litke has posted comments on this change.

Change subject: added support for controlling iotune params
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.ovirt.org/#/c/31251/3/mom/HypervisorInterfaces/vdsmInterface.py
File mom/HypervisorInterfaces/vdsmInterface.py:

Line 217:             vm = API.VM(uuid)
Line 218:             response = vm.getStats()
Line 219:             self._check_status(response)
Line 220: 
Line 221:             ioTuneStats = response['statsList'][0].get('ioTune', None)
Has this API been approved in upstream vdsm yet?
Line 222:             for ioTune in ioTuneStats:  # convert from strings back 
to integers
Line 223:                 ioTune['ioTune'] = dict((k, self._to_int(v)) for k, v 
in
Line 224:                                         ioTune['ioTune'].iteritems())
Line 225: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic44e0780bbcc808fe057c378ac83af4d7729a2fb
Gerrit-PatchSet: 3
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Jiří Moskovčák <jmosk...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmosk...@redhat.com>
Gerrit-Reviewer: Martin Sivák <msi...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to