Dan Kenigsberg has submitted this change and it was merged. Change subject: vmstats: do not report empty balloonInfo ......................................................................
vmstats: do not report empty balloonInfo Do not report empty value for 'balloonInfo' stats, much like we do for 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: I146e1f476434f023281f475ffaaa808f4cdf862b Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/43539 Reviewed-by: Martin Polednik <[email protected]> Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/virt/vmstats.py 1 file changed, 3 insertions(+), 1 deletion(-) 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/43539 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I146e1f476434f023281f475ffaaa808f4cdf862b Gerrit-PatchSet: 13 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
