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

Default to INFO level #214

Merged
merged 1 commit into from Feb 5, 2019

Conversation

Projects
None yet
7 participants
@patrobinson
Copy link
Contributor

patrobinson commented Feb 5, 2019

The default log level is not sane for production systems. Services have been known to produce more than 50GB of logs per day with DEBUG level enabled.

Default to INFO level
The default log level is not sane for production systems. Services have been known to produce more than 50GB of logs per day with DEBUG level enabled.
@samuelsuarezsanchez
Copy link

samuelsuarezsanchez left a comment

I think INFO is a sensible default level of logging. Overwriting it is also trivial, so 👍 to this.

@stevehodgkiss
Copy link
Member

stevehodgkiss left a comment

👍

@zubin

zubin approved these changes Feb 5, 2019

@mjward

mjward approved these changes Feb 5, 2019

@orien

orien approved these changes Feb 5, 2019

@patrobinson patrobinson merged commit 89ccf9f into master Feb 5, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@patrobinson patrobinson deleted the default-to-info branch Feb 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment