-
Notifications
You must be signed in to change notification settings - Fork 699
OTEL_PROPAGATORS does not support "none" value #4143
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
Comments
I would like to work on this issue |
Hi @xrmx, |
I believe @Rajamanosankari has already been assigned to this issue. Feel free to reach out to them if you'd like to collaborate or take over. |
@lzchen I am an Outreachy contributor for project. This is my first contribution to otel-python repo. If @Rajamanosankari isn't working can I take on this issue? Thanks |
Seeing as @Rajamanosankari has not responded in the thread I will reassign. |
Describe your environment
OS: Linux/amd64
Python version: Python 3.12
SDK version: I installed following libs using pip, so I assume the latest version (
opentelemetry-exporter-otlp opentelemetry-instrumentation-requests opentelemetry-instrumentation-flask
)API version: I installed following libs using pip, so I assume the latest version (
opentelemetry-exporter-otlp opentelemetry-instrumentation-requests opentelemetry-instrumentation-flask
)What happened?
It seems that I can't disable context propagation in Python by setting the
OTEL_PROPAGATORS
ENV variable tonone
as mentioned here.When trying to do so, I get the following error:
It seems that error happens at this line and it's because we are missing the none propagator definition here in opentelemetry-api/pyproject.toml. (I am not a python expert, so my assumption can be wrong)
Steps to Reproduce
OTEL_PROPAGATORS=none
Expected Result
No errors are expected, and automatic propagation is disabled.
Actual Result
Error that the
none
propagator is not defined.Additional context
I am not sure if setting the
OTEL_PROPAGATORS=""
will be valid, but according to the documentation,none
should be a valid value.Would you like to implement a fix?
None
The text was updated successfully, but these errors were encountered: