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

Switch logging to promlog library. #21

Merged
merged 4 commits into from
Dec 10, 2021
Merged

Conversation

woblerr
Copy link
Owner

@woblerr woblerr commented Dec 4, 2021

This is the first step to implementing the TLS and basic user authentication option #19 .

  • The logging mechanism is switched to promlog library from prometheus/common.
  • The following flags have been added:
    • --log.level - Only log messages with the given severity or above. One of: [debug, info, warn, error]. Default value: info.
    • --log.format - Output format of log messages. One of: [logfmt, json]. Default value: logfmt.
    • -h - Short variation of --help.
  • Added printing of values and labels for metrics for debug level.

Use ptomlog package for logging messages.
Add flags --log.level and --log.format from promlog package.
Add short help flag -h.
@woblerr woblerr self-assigned this Dec 4, 2021
@woblerr woblerr marked this pull request as ready for review December 6, 2021 17:19
@woblerr woblerr merged commit f3a332d into master Dec 10, 2021
@woblerr woblerr deleted the add_tls_and_basic_auth branch December 10, 2021 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant