[BUG] Make opentelemetry dependencies optional #3290
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of changes
This is just a DRAFT meant to discuss what it would take to provide a workaround for #2571 and any other potential issues with our protobuf files being inconsistent. The primary question we need to answer is:
In other words, this is imperfect, so is it even worth doing versus making OTEL truly optional in our codebase by doing a huge refactor.
Summarize the changes made by this PR.
opentelemetry
dependencies optionalpip install chromadb[opentelemetry]
Test plan
I tested that I can target the optional dependencies by doing
pip install '.[opentelemetry]'
Tests pass locally with
pytest
for python,yarn test
for js,cargo test
for rustDocumentation Changes