Yaniv Bronhaim has posted comments on this change. Change subject: Adding report_stats to virt.stats ......................................................................
Patch Set 14: (1 comment) https://gerrit.ovirt.org/#/c/59066/14/lib/vdsm/virt/vmstats.py File lib/vdsm/virt/vmstats.py: Line 189: else: Line 190: logging.error('Failed to get VM cpu count') Line 191: Line 192: Line 193: def report_stats(vms_stats): > I don't see any use of this - I assume this is preparation for another patc see following patch https://gerrit.ovirt.org/#/c/59067 Line 194: report = {} Line 195: try: Line 196: for vm_uuid in vms_stats: Line 197: prefix = "vms." + vm_uuid -- To view, visit https://gerrit.ovirt.org/59066 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idf494c6a3087d04c12731c587b619f253dd51165 Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org