Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Python auto instrumentation fails with current version #1148

Closed
aabmass opened this issue Oct 7, 2022 · 3 comments · Fixed by #1149
Closed

Python auto instrumentation fails with current version #1148

aabmass opened this issue Oct 7, 2022 · 3 comments · Fixed by #1149
Labels
area:auto-instrumentation Issues for auto-instrumentation auto-instrumentation:python

Comments

@aabmass
Copy link
Member

aabmass commented Oct 7, 2022

With a very basic Python app, injection causes the auto instrumentation to fail with RuntimeError: Requested component 'otlp_proto_grpc' not found in entry points for 'opentelemetry_metrics_exporter' . See open-telemetry/opentelemetry-python-contrib#1361, but I'm happy to provide a repro as well.

What's happening is the default value for OTEL_METRICS_EXPORTER is otlp_proto_grpc, but that exporter is not included in the image's requirements.txt as the operator should be using HTTP.

Until we implement an OTLP/HTTP exporter for python that can be used in the operator (open-telemetry/opentelemetry-python#2447) I think we should just disable metrics in the operator by setting OTEL_METRICS_EXPORTER=none.
@aabmass aabmass mentioned this issue Oct 7, 2022
Merged
@pavolloffay
Copy link
Member

+1 on the approach.

would you like to submit a PR?

@pavolloffay
Copy link
Member

@aabmass could you please book an issue in this repository to set the metrics exporter to HTTP once done in the SDK?

@pavolloffay pavolloffay reopened this Oct 11, 2022
@aabmass aabmass mentioned this issue Oct 13, 2022
Closed
@aabmass
Copy link
Member Author

aabmass commented Oct 13, 2022

Done #1168

@aabmass aabmass closed this as completed Oct 13, 2022
@mco-gh mco-gh mentioned this issue Apr 10, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:auto-instrumentation Issues for auto-instrumentation auto-instrumentation:python
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Set OTEL_METRICS_EXPORTER to none to prevent using the default value aabmass/opentelemetry-operator
2 participants
@aabmass @pavolloffay