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

[Feature/Operator] Configurable log levels #20

Merged
merged 3 commits into from
Mar 16, 2022

Conversation

juldrixx
Copy link
Contributor

@juldrixx juldrixx commented Mar 15, 2022

Q A
Bug fix? no
New feature? yes
API breaks? no
Deprecations? no
Related tickets Orange-OpenSource/nifikop#181
License Apache 2.0

What's in this PR?

Added adjustable logging level for the operator. (All levels supported by zapcore are included.)

Why?

We can produce less logs, especially if we know that our setup is working, and concentrate on errors/warnings in the logs.

Checklist

  • Implementation tested
  • Error handling code meets the guideline
  • Logging code meets the guideline
  • User guide and development docs updated (if needed)
  • Append changelog with changes

@juldrixx juldrixx requested a review from erdrix March 15, 2022 21:56
@erdrix erdrix merged commit 6f6f56e into master Mar 16, 2022
@erdrix erdrix deleted the feature/configurable_log_levels branch March 16, 2022 07:50
cannonpalms referenced this pull request in influxdata/nifikop Mar 23, 2022
erdrix added a commit that referenced this pull request Aug 3, 2022
* Add configurable log levels

* Update changelog

Co-authored-by: Alexandre Guitton <[email protected]>
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