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.

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

⬆️♻️ Update open telemetry dependencies #282

Merged
merged 4 commits into from
Jan 21, 2025

Conversation

evaline-ju
Copy link
Collaborator

@evaline-ju evaline-ju commented Jan 17, 2025

Updating to the latest open telemetry-related crates, particularly centering around use of opentelemetry 0.27.0

  • The main interface changes are to the pipeline concept, instead switching to explicit use of exporters and providers. Changes are noted in the opentelemetry_otlp v0.27.0 update , particularly with the changes from https://github.com/open-telemetry/opentelemetry-rust/pull/2221. The protocol (grpc vs. http) is only needed to determine the exporter, and the providers are built after with the exporter.
  • The Deprecated Config public methods will show up as warnings so these were also updated, where resource and sampler move to the providers.

Closes: #274

Signed-off-by: Evaline Ju <[email protected]>
Copy link
Collaborator

@declark1 declark1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@evaline-ju evaline-ju merged commit cf5dd83 into foundation-model-stack:main Jan 21, 2025
2 checks passed
@evaline-ju evaline-ju deleted the otel-updates branch January 21, 2025 19:49
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.

Update dependencies and drop unused dependencies
2 participants