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

logging: enable subsystem warn level by default #565

Merged
merged 2 commits into from
Jun 13, 2024

Conversation

davidweisse
Copy link
Contributor

@davidweisse davidweisse commented Jun 12, 2024

This enables the all log subsystems at warn level or above. These logs will be get printed regardless of the CONTRAST_LOG_SUBSYSTEMS list. The CONTRAST_LOG_LEVEL variable is also considered, so when the global log level is set to error, only error messages will get printed from the subsystems

@davidweisse davidweisse added the no changelog PRs not listed in the release notes label Jun 12, 2024
@davidweisse davidweisse requested a review from katexochen as a code owner June 12, 2024 15:51
@davidweisse davidweisse force-pushed the dav/activate-snp-logging branch from aa29f85 to 743f383 Compare June 12, 2024 16:06
@burgerdev
Copy link
Contributor

@katexochen I wonder if we should enable all log subsystems at WARN level by default, and then ensure some rigour so that actually relevant messages (and only those) are reported at that level.

@katexochen
Copy link
Member

@katexochen I wonder if we should enable all log subsystems at WARN level by default, and then ensure some rigour so that actually relevant messages (and only those) are reported at that level.

Fine for me! :)

@davidweisse davidweisse force-pushed the dav/activate-snp-logging branch from 743f383 to 2ecf136 Compare June 13, 2024 09:54
@katexochen katexochen added changelog PRs that should be part of the release notes and removed no changelog PRs not listed in the release notes labels Jun 13, 2024
@katexochen
Copy link
Member

@davidweisse could you update PR title and description?

@davidweisse davidweisse changed the title coordinator: enable snp validator logging by default logging: enable subsystem warn level by default Jun 13, 2024
@davidweisse davidweisse merged commit 30cee6a into main Jun 13, 2024
10 checks passed
@davidweisse davidweisse deleted the dav/activate-snp-logging branch June 13, 2024 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog PRs that should be part of the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants