From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: MOM: Reduce log messages when MOM unavailable
......................................................................
MOM: Reduce log messages when MOM unavailable
When MOM is not available the log gets cluttered with messages.
This patch utilizes the throttledlog module to display the messages
once every 100 calls instead of every time the methods are called.
Change-Id: I0af0d3dc694d16fae791bb5af2feb5c7b6deeb23
Bug-Url:
https://bugzilla.redhat.com/1437341
Signed-off-by: Jenny Tokar <jtokar(a)redhat.com>
---
M lib/vdsm/momIF.py
M lib/vdsm/throttledlog.py
M tests/throttledlog_test.py
3 files changed, 20 insertions(+), 2 deletions(-)
Approvals:
Denis Chaplygin: Looks good to me, but someone else must approve
Martin Sivák: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Looks good to me, but someone else must approve
Jenny Tokar: Verified
--
To view, visit
https://gerrit.ovirt.org/75681
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0af0d3dc694d16fae791bb5af2feb5c7b6deeb23
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>