Skip to content

bump: Update opentelemetry version - #26

Merged
RasmusRendal merged 1 commit into
mainfrom
RasmusRendal/bump_opentelemetry
Aug 3, 2026
Merged

bump: Update opentelemetry version#26
RasmusRendal merged 1 commit into
mainfrom
RasmusRendal/bump_opentelemetry

Conversation

@RasmusRendal

@RasmusRendal RasmusRendal commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

Bumps all the opentelemetry crates to version 0.32 of the SDK.

This involves some changes to opentelemetry_instrumentation_tower
(Previously known as tower_otel_http_metrics). The only visible change is
that when adding a metrics layer, the service_name is no longer required,
as the global meter is now always used by default.

I verified that all dependencies use the correct version of the SDK by
running cargo tree --duplicates -p opentelemetry

@RasmusRendal
RasmusRendal requested a review from a team as a code owner February 4, 2026 13:40
@RasmusRendal
RasmusRendal marked this pull request as draft February 4, 2026 13:43
@RasmusRendal

RasmusRendal commented Feb 4, 2026

Copy link
Copy Markdown
Contributor Author

They moved tower-otel-http-metrics to opentelemetry-rust-contrib. In the move, they seem to have forgotten to update it with the new opentelemetry release. We'll have to wait for opentelemetry-instrumentation-tower to end up on crates.io.

Someone uploaded a opentelemetry-instrumentation-tower release to crates.io, but I'd prefer to wait until https://crates.io/teams/github:open-telemetry:rust-publishers has ownership of this crate before we start using it.

The upstream issue related to the crate being published is open-telemetry/opentelemetry-rust-contrib#619

@RasmusRendal
RasmusRendal force-pushed the RasmusRendal/bump_opentelemetry branch from 081ab9f to 3299542 Compare June 18, 2026 08:56
@RasmusRendal
RasmusRendal force-pushed the RasmusRendal/bump_opentelemetry branch 4 times, most recently from 78bb56f to 9ddd9e6 Compare August 3, 2026 06:31
Bumps all the opentelemetry crates to version 0.32 of the SDK.

This involves some changes to `opentelemetry_instrumentation_tower`
(Previously known as `tower_otel_http_metrics`). The only visible change is
that when adding a metrics layer, the `service_name` is no longer required,
as the global meter is now always used by default.

I verified that all dependencies use the correct version of the SDK by
running `cargo tree --duplicates -p opentelemetry`
@RasmusRendal
RasmusRendal force-pushed the RasmusRendal/bump_opentelemetry branch from 9ddd9e6 to c8bd580 Compare August 3, 2026 06:39
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.83%. Comparing base (97385db) to head (c8bd580).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/axum.rs 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
+ Coverage   74.18%   75.83%   +1.65%     
==========================================
  Files           5        5              
  Lines         275      269       -6     
==========================================
  Hits          204      204              
+ Misses         71       65       -6     
Files with missing lines Coverage Δ
src/axum.rs 0.00% <0.00%> (ø)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 97385db...c8bd580. Read the comment docs.

@RasmusRendal
RasmusRendal marked this pull request as ready for review August 3, 2026 06:44
@RasmusRendal
RasmusRendal merged commit c8bd580 into main Aug 3, 2026
8 of 9 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.

2 participants