Skip to content

Logging API hide std_to_otel function to convert python logging severity to otel severity #4649

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

hectorhdzg
Copy link
Member

Description

Hide unexpected method in Logs API, this is only being used internally by LoggerHandler

Fixes #4314

Type of change

Please delete options that are not relevant.

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@hectorhdzg hectorhdzg requested a review from a team as a code owner June 19, 2025 23:55
Copy link
Contributor

@tammy-baylis-swi tammy-baylis-swi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, confirmed that std_to_otel isn't used anywhere in contrib either. I'm pretty sure the aio-pika test failure is unrelated.

Copy link
Member

@emdneto emdneto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

During a quick search on GitHub, I found this. It looks like a few repositories are already using it, but I’d rather deal with breakages now than after the signal is marked as stable. Would like to have some thoughts from maintainers.

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.

Logs API: hide or rename std_to_otel function to convert python logging severity to otel severity
3 participants