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

dev: adjust prometheus log level #2170

Closed
wants to merge 1 commit into from
Closed

dev: adjust prometheus log level #2170

wants to merge 1 commit into from

Conversation

dctalbot
Copy link
Contributor

  • Identified the issue which this PR solves.
  • Read the CONTRIBUTING document.
  • Code builds clean without any errors or warnings.
  • Added appropriate tests for any new functionality.
  • All new and existing tests passed.
  • Added comments in the code, where necessary.
  • Ran make check to catch common errors. Fixed any that came up.

Description:
This PR sets the Prometheus log severity threshold to "warn"

Which issue(s) this PR fixes:
Suggested in #2156

Additional Info:
For some reason, this flag can only be set on the CLI (not config file).
Options were debug, info, warn, and error.

I decided to keep prod using the default (info) since the main motivation for this change is DX.

@dctalbot
Copy link
Contributor Author

superseded by #2172

@dctalbot dctalbot closed this Feb 16, 2022
@dctalbot dctalbot deleted the prom-log branch March 14, 2022 14:46
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.

1 participant