Francesco Romani has posted comments on this change. Change subject: Collect current QoS settings for IO devices and report through RunningVmStats ......................................................................
Patch Set 6: (2 comments) http://gerrit.ovirt.org/#/c/28712/6/lib/vdsm/config.py.in File lib/vdsm/config.py.in: Line 189: ('vm_sample_vcpu_pin_window', '2', None), Line 190: Line 191: ('vm_sample_cpu_tune_interval', '15', None), Line 192: Line 193: ('vm_sample_io_tune_interval', '15', None), Since we need to parse the full domain XML at each run ( :( ), any chance to make this less frequently? Let's pick the longest safe interval. Line 194: Line 195: ('trust_store_path', '@TRUSTSTORE@', Line 196: 'Where the certificates and keys are situated.'), Line 197: http://gerrit.ovirt.org/#/c/28712/6/vdsm/virt/vm.py File vdsm/virt/vm.py: Line 341: self._log.exception("Error while collecting ioTune for %s", Line 342: self._vm.name) Line 343: return None Line 344: Line 345: devxml = _domParseStr(lastXMLDesc).childNodes[0] \ > Unfortunately, I do not see any other way to get the up-to-date and accurat Yep, I thought this is the safest route. Not your fault, but the consequence of a bad API. Still, this is bad for performance. We need to take countermeasures, and maybe file some libvirt bugs. Line 346: .getElementsByTagName('devices')[0] Line 347: Line 348: diskxml = devxml.getElementsByTagName('disk') Line 349: -- To view, visit http://gerrit.ovirt.org/28712 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie4fab132b60ce6ae2285115034f3d1ffcdca9e9d Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msi...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Martin Sivák <msi...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
