Dan Kenigsberg has submitted this change and it was merged. Change subject: vmstats: do not report empty ioTune ......................................................................
vmstats: do not report empty ioTune Do not report empty value for 'ioTune' stats, much like we do for balloonInfo, disks and networks In general, we should avoid to report empty values if the emptiness carries no meaning (noteworthy counterexample: vmJobs for live merge). Change-Id: I1c34e2ffb1ebeb500be22d6db5f5a8ea2c1bb19f Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/43596 Reviewed-by: Martin Polednik <[email protected]> Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/virt/vmstats.py 1 file changed, 9 insertions(+), 6 deletions(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/43596 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c34e2ffb1ebeb500be22d6db5f5a8ea2c1bb19f Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
