-
Notifications
You must be signed in to change notification settings - Fork 765
Bump opentelemetry to 0.29 #11552
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
Bump opentelemetry to 0.29 #11552
Conversation
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.
We cannot yet merge as is, as this now pulls in two versions of opentelemetry. We will need to upgrade our version of opentelemetry first to match.
8206 tests run: 7767 passed, 0 failed, 439 skipped (full report)Flaky tests (6)Postgres 15
Postgres 14
Code coverage* (full report)
* collected from Rust tests only The comment gets automatically updated with the latest test results
6c6fe25 at 2025-04-16T17:39:18.955Z :recycle: |
# This revision uses opentelemetry 0.27. There's no tag for it. | ||
tower-otel = { git = "https://github.com/mattiapenati/tower-otel", rev = "56a7321053bcb72443888257b622ba0d43a11fcd" } | ||
|
||
tower-otel = { version = "0.5.0", features = ["axum"] } |
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.
default-features = false
?
This rebuilds #11552 on top the current Cargo.lock. --------- Co-authored-by: Conrad Ludgate <conradludgate@gmail.com>
Crate
tower-otel
version 0.5.0 has been released. This version uses the newest release of opentelemetry and support for axum has been included.The dependency has been updated to v0.5.0, so now neon does not rely on a specific revision of the crates but on a tagged release.
Upgrading opentelemetry to 0.29 has required quite a few changes.