Nir Soffer has posted comments on this change. Change subject: Adding report_stats to host.api ......................................................................
Patch Set 10: (1 comment) https://gerrit.ovirt.org/#/c/58661/10/lib/vdsm/host/api.py File lib/vdsm/host/api.py: Line 132: report[net_prefix + '.rx'] = if_info['rx'] Line 133: report[net_prefix + '.tx'] = if_info['tx'] Line 134: Line 135: reports.send(report) Line 136: except KeyError as e: pyflakes is not happy: 08:29:09 ./lib/vdsm/host/api.py:136: local variable 'e' is assigned to but never used Line 137: logging.exception('Report host stats failed') Line 138: Line 139: Line 140: def _readSwapTotalFree(): -- To view, visit https://gerrit.ovirt.org/58661 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie65c865669422b7652b028492b1380157fc5e177 Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Irit Goihman <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
