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
Fix log levels #10325
Fix log levels #10325
Conversation
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve 🏎️
LGTM label has been added. Git tree hash: 54ab1e0ec26c93d92147378072ecd14180e7e839
|
/retest Review the full test history Silence the bot with an |
/override pre-kubermatic-test-integration |
@xrstf: Overrode contexts on behalf of xrstf: pre-kubermatic-test-integration In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest |
What does this PR do / Why do we need it:
We should use the INFO log level only for actionable messages, like when we actually created/deleted a resource. There is little use (IMHO) to log "Skipping reconcile for StatefulSets, is not healthy yet" over and over again. So this PR changes all log statements that used INFO to DEBUG if they only log non-essential progress information.
I also, randomly, replaced Eventf() with Event() where we don't actually do string formatting.
Does this PR introduce a user-facing change?: