Yaniv Bronhaim has submitted this change and it was merged. Change subject: iface_sampling: Fix debug logging samples ......................................................................
iface_sampling: Fix debug logging samples Currently we have a debug logging entry whenever the tx/rx rates go over 100%. In this log, we try to help debugging of the issue by printing the sample value of rx and tx for all the samples in the AVERAGING_WINDOW (size 5). The problem comes from the fact that the decision to log is taken by checking samples hs0 and hs1 iterating over all the devices existing on the older hs1. However, there is no guarantee that the devices would exist in hs2, hs3, ..., hs_AVERAGING_WINDOW. The fix is to simply not include the samples for which the device is missing. Change-Id: Ibaa80f5fe069875d8e48deb1783f102b2da8f39d Bug-Url: https://bugzilla.redhat.com/1116577 Signed-off-by: Antoni S. Puimedon <[email protected]> Reviewed-on: http://gerrit.ovirt.org/29916 Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-by: Francesco Romani <[email protected]> Tested-by: Dan Kenigsberg <[email protected]> Reviewed-on: http://gerrit.ovirt.org/31789 Reviewed-by: Genadi Chereshnya <[email protected]> Reviewed-by: Yaniv Bronhaim <[email protected]> --- M vdsm/virt/sampling.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Genadi Chereshnya: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, approved Antoni Segura Puimedon: Verified -- To view, visit http://gerrit.ovirt.org/31789 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibaa80f5fe069875d8e48deb1783f102b2da8f39d Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Genadi Chereshnya <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
