Skip to content

Upgrade open telemetry #403

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

Merged

Conversation

mdevino
Copy link
Collaborator

@mdevino mdevino commented May 22, 2025

Closes #354.

Based myself on migration guides to 0.28 and 0.29.

Changes summary:

  • Renamed TracerProvider to SdkTracerProvider.
  • Updated Resource creation (using builder pattern).
  • No longer need to send runtime::Tokio as a parameter.
  • shutdown() now must be called from the providers, not from global.
  • Created two new error variants - BuilderError and ShutdownError.

@mdevino mdevino marked this pull request as ready for review May 27, 2025 17:18
Copy link
Collaborator

@evaline-ju evaline-ju left a comment

Choose a reason for hiding this comment

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

Looks like there's some conflicts here, likely from #404

@mdevino mdevino force-pushed the upgrade_open_telemetry branch from 8cfcd33 to e826c1a Compare June 2, 2025 17:30
@mdevino
Copy link
Collaborator Author

mdevino commented Jun 2, 2025

@evaline-ju Thanks. I've just pushed a rebase. Just not sure if merging this prior to #391 and #393 will cause any conflicts for them as those seem more complex.

Copy link
Collaborator

@evaline-ju evaline-ju left a comment

Choose a reason for hiding this comment

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

LGTM

mdevino added 2 commits June 3, 2025 17:24
@mdevino mdevino force-pushed the upgrade_open_telemetry branch from e826c1a to 8737a6b Compare June 3, 2025 20:24
@evaline-ju evaline-ju merged commit efa5067 into foundation-model-stack:main Jun 3, 2025
2 checks passed
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.

Upgrade open telemetry dependencies
2 participants