-
Notifications
You must be signed in to change notification settings - Fork 45
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
Don't use pkg_resources.get_distribution(..).version
#256
Conversation
/gcbrun |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also update here?
opentelemetry-operations-python/opentelemetry-exporter-gcp-trace/tests/test_cloud_trace_exporter.py
Line 98 in 8f8d2f0
pkg_resources.get_distribution("opentelemetry-sdk").version |
Thanks for noticing that. I also found I missed another use in |
372a1a9
to
8ef25e2
Compare
Fix in GoogleCloudPlatform#254 didn't apply to tox environments that run `pip install .` `test_detects_gce` was failing with opentelemetry-sdk version 1.19.0 instead of 1.18.0
Similar to discussion at googleapis/python-api-core#27 I am using opentelemetry in an environment where modules are vendored without pip. So, pkg_resources can not be used to search for a module at runtime to get its version.
/gcbrun |
/gcbrun |
Similar to discussion at googleapis/python-api-core#27 I am using opentelemetry in an environment where modules are vendored without pip. So, pkg_resources can not be used to search for a module at runtime to get its version.