Skip to content

Clarify OTEL_LOG_LEVEL is to control internal logging#4267

Merged
reyang merged 1 commit intoopen-telemetry:mainfrom
cijothomas:cijothomas/clarify-sdk-log
Oct 21, 2024
Merged

Clarify OTEL_LOG_LEVEL is to control internal logging#4267
reyang merged 1 commit intoopen-telemetry:mainfrom
cijothomas:cijothomas/clarify-sdk-log

Conversation

@cijothomas
Copy link
Copy Markdown
Member

@cijothomas cijothomas commented Oct 21, 2024

Changes

Trying to make it more obvious that OTEL_LOG_LEVEL is to control SDK's own internal logging level. (As opposed to controlling the levels in LogBridge)

For non-trivial changes, follow the change proposal process.

@cijothomas cijothomas requested review from a team October 21, 2024 02:58
@reyang reyang added the editorial Editorial changes only (typos, changelog, ...). No content-related changes of any kind. label Oct 21, 2024
@reyang reyang merged commit d0b2693 into open-telemetry:main Oct 21, 2024
@cijothomas cijothomas deleted the cijothomas/clarify-sdk-log branch October 21, 2024 16:58
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
kaylareopelle added a commit to kaylareopelle/opentelemetry-specification that referenced this pull request Oct 1, 2025
When reading through open-telemetry#4267, I noticed that the Probability Sampler description still had a reference to ratio-based. I think that entire section may be cut since the clarifications were specifically related to the old name.

Also, found a small typo.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial Editorial changes only (typos, changelog, ...). No content-related changes of any kind.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants