-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Setting -log-level debug
introduces client-go
logging at the INFO
level
#12887
Comments
The issue stems from us passing logrus' standard logger to klog, which uses an InfoLevel by default. We could easily change that level to Debug, and have all client-go log entries show as debug in the output. |
mikutas
added a commit
to mikutas/linkerd2
that referenced
this issue
Jul 30, 2024
Fixes linkerd#12887 Signed-off-by: Takumi Sue <[email protected]>
mikutas
added a commit
to mikutas/linkerd2
that referenced
this issue
Jul 30, 2024
Fixes linkerd#12887 Signed-off-by: Takumi Sue <[email protected]>
mikutas
added a commit
to mikutas/linkerd2
that referenced
this issue
Jul 31, 2024
Fixes linkerd#12887 Signed-off-by: Takumi Sue <[email protected]>
mikutas
added a commit
to mikutas/linkerd2
that referenced
this issue
Jul 31, 2024
Fixes linkerd#12887 Signed-off-by: Takumi Sue <[email protected]>
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What is the issue?
Setting
-log-level debug
on the various Go Control Plane components enables more-detailedclient-go
logging, which is good, except those log lines are printed at theINFO
level.How can it be reproduced?
Logs, error output, etc
see above
output of
linkerd check -o short
n/a
Environment
edge-24.4.5
Possible solution
client-go logs should show up at the
DEBUG
level. more generally: setting-log-level debug
should introduceDEBUG
log lines, not moreINFO
lines.Additional context
No response
Would you like to work on fixing this bug?
maybe
The text was updated successfully, but these errors were encountered: