Skip to content

Set logger level before tracing initialization#428

Closed
uzzz wants to merge 1 commit into
envoyproxy:mainfrom
uzzz:main
Closed

Set logger level before tracing initialization#428
uzzz wants to merge 1 commit into
envoyproxy:mainfrom
uzzz:main

Conversation

@uzzz

@uzzz uzzz commented Jun 5, 2023

Copy link
Copy Markdown

An issue: if JSON log format is used InitProductionTraceProvider prints the time="2023-06-05T15:29:34Z" level=info msg="TracerProvider initialized with following parameters: ... line in the default (text) format.
Fix: set log level before initializing OTEL.

Signed-off-by: Maksim Zhylinski <maksim@zhylinski.me>
@uzzz uzzz changed the title Set logger level before tracing Set logger level before tracing initialization Jun 8, 2023
@github-actions

github-actions Bot commented Jul 8, 2023

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions Bot added the stale label Jul 8, 2023
@github-actions

Copy link
Copy Markdown

This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant