Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

log: add leveled logging #366

Closed
wants to merge 3 commits into from
Closed

log: add leveled logging #366

wants to merge 3 commits into from

Conversation

magiconair
Copy link
Contributor

This patch implements a simple leveled logging output to provide the
ability to have DEBUG logs which are not logged by default. The
implementation allows changing the log level at runtime but in this
implementation there is no mechanism to change the value at runtime.

This patch implements a simple leveled logging output to provide the
ability to have DEBUG logs which are not logged by default. The
implementation allows changing the log level at runtime but in this
implementation there is no mechanism to change the value at runtime.
@magiconair magiconair self-assigned this Oct 7, 2017
@CLAassistant
Copy link

CLAassistant commented Oct 7, 2017

CLA assistant check
All committers have signed the CLA.

@magiconair
Copy link
Contributor Author

Still need to add a test for the LevelWriter

@magiconair magiconair closed this in 5351006 Oct 9, 2017
magiconair added a commit that referenced this pull request Oct 9, 2017
When a cloud environment has nodes and/or services in maintenance then
fabio can generate excessive logging every time the routing table is
generated. Since these messages can be useful they remain in the code
but are being logged as DEBUG which after merging #366 is no longer
logged by default.

Fixes #367
@magiconair magiconair deleted the leveled-logging branch October 9, 2017 00:45
@magiconair magiconair added this to the 1.6.0 milestone Oct 10, 2017
@magiconair magiconair modified the milestones: 1.6, 1.5.3 Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants