Repository navigation
chore(deps)(deps): bump opentelemetry_sdk from 0.28.0 to 0.32.1 - #446
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [opentelemetry_sdk](https://github.com/open-telemetry/opentelemetry-rust) from 0.28.0 to 0.32.1. - [Release notes](https://github.com/open-telemetry/opentelemetry-rust/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-rust/blob/main/docs/release_0.32.md) - [Commits](https://github.com/open-telemetry/opentelemetry-rust/commits) --- updated-dependencies: - dependency-name: opentelemetry_sdk dependency-version: 0.32.1 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
workspace-hack and Cargo.lock regenerated via cargo hakari generate. tonic 0.12.3 / prost 0.13.5 remain in the lock as opentelemetry-otlp 0.28's own nodes (until Dependabot #446); the workspace family is on 0.14 throughout. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Deferring —
|
| tracing-opentelemetry | requires |
|---|---|
| 0.29 (ours) | opentelemetry, opentelemetry_sdk, opentelemetry-otlp ^0.28 |
| 0.33 | all of them ^0.32 |
And the jump reaches past otel into the gRPC stack:
| ours (0.28) | 0.32 | |
|---|---|---|
tonic |
^0.12.3 |
^0.14.1 |
prost |
^0.13 |
^0.14 |
So the real unit of work is: opentelemetry + opentelemetry_sdk +
opentelemetry-otlp → 0.32, tracing-opentelemetry → 0.33, and the
tonic/prost stack → 0.14 — which is #1029 (tonic 0.12.3 → 0.14.6), a
substantial migration of its own since tonic 0.14 splits codegen into
tonic-prost.
Worth flagging: #1033 (tracing-opentelemetry 0.29 → 0.33) is the same
bump seen from the other end and fails the same way — its lockfile also
carries opentelemetry 0.28 and 0.32. Neither #446 nor #1033 can go green
alone, and no PR is open for opentelemetry or opentelemetry-otlp at all,
so even landing both together would still leave the family split.
The decision it turns on
Sequencing: tonic 0.14 (#1029) has to land before the otel family can move,
because otlp 0.32 hard-requires it. That makes this a scheduled,
coordinated bump, not a bump-and-fix.
The concrete follow-up I would suggest is a dependabot group, because this
repo already made exactly this call for the same reason —
.github/dependabot.yml says of the grpc group:
the tonic/prost family is version-locked to itself … Individual major PRs
for these crates can NEVER go green — see #25 / #26 / #29 — so "one
crate at a time" is not a reviewable unit here, it is an unbuildable one.
The otel family is that situation verbatim and has no group. Adding
opentelemetry, opentelemetry_sdk, opentelemetry-otlp,
opentelemetry-*, tracing-opentelemetry as a group would make this arrive
as one landable PR instead of two permanently-red ones. I did not make that
config change here: it is a policy edit to dependabot.yml, not this
branch's dependency, and folding it in would break the one-dependency-per-
branch rule.
Why I am NOT adding an ignore
I deliberately did not post @dependabot ignore this major version, even
though this PR will keep reappearing.
An ignore is scoped to the crate and version, not to the PR. Ignoring
opentelemetry_sdk 0.32 would also exclude it from the future grouped otel
PR — that is, it would suppress the crate that the fix depends on and quietly
make the correct bump impossible to assemble. The reappearing PR is noise;
sabotaging the remedy is worse than noise. The grouping change above is the
thing that stops the churn without that side effect.
State of the branch
No source change and nothing pushed. I rebased locally to reproduce against
current main, but the 435-commit-stale Cargo.lock conflicts, and there is
no reason to spend a CI run on a bump that cannot go green — so Dependabot
keeps the branch. The failing lanes above are from June and are still
accurate as to cause.
Third family with the same structural lock as tonic/prost and sha2/hmac. The OTel Rust crates ship as one release set: the SDK and exporter crates depend on an exact `opentelemetry` core minor, and tracing-opentelemetry runs one minor ahead of the core it targets (0.29 -> core 0.28, 0.33 -> core 0.32). So a single-crate bump puts two cores in the graph at once. #446 bumps opentelemetry_sdk 0.28 -> 0.32.1 on its own; that release requires `opentelemetry 0.32.0`, while the workspace still declares opentelemetry and opentelemetry-otlp at 0.28. Its lockfile ends up carrying opentelemetry 0.28.0 AND 0.32.0, so the SDK being configured speaks a different API than the exporter consuming it — which is why that PR fails clippy, check, tests and the musl cross lane together rather than failing one of them. #1033 (tracing-opentelemetry 0.29 -> 0.33) is the mirror image. Group them at every update level, and add the patterns to cargo-minor-patch's exclude list so first-match ordering cannot peel one crate off the set.
Deferred — unbuildable alone; the OTel crates are one release setTriaged in today's sweep. This is not "the migration is hard" — as written, this What the failures actually are
Root causeThe OpenTelemetry Rust crates ship as a single release set. The SDK and exporter But the workspace still declares the rest of the family at 0.28 opentelemetry = "0.28"
opentelemetry_sdk = { version = "0.32", features = ["rt-tokio"] } # <- this PR
opentelemetry-otlp = { version = "0.28", features = ["grpc-tonic", "trace"] }
tracing-opentelemetry = "0.29"So the resolved graph carries both cores at once: The SDK being configured speaks a different #1033 ( Why I did not just fold them togetherOne dependency per branch. Folding
Done this run#1046 adds an Once that lands, Dependabot proposes the whole family as one PR — the first Re-check conditionSuperseded, not timed:
Also worth knowingThis branch is 441 commits behind |
Closing — superseded by #1060, and the blocker I named in August is goneRe-checked this in today's sweep, as the earlier deferral required. Both The stated blocker no longer existsThe August comment deferred this on "the tonic/prost stack → 0.14 — which is So the prerequisite is in. otel 0.32 was genuinely unlandable before it — a But this PR is still the wrong unit of workThat deferral's other half was right: #1060 is that unit — It is also a net simplification: main currently carries two tonics For the record, #1060 is not blocked on a coupling at all. Its one failure is DispositionClosing as superseded. #1033 ( Follow #1060. |
Pull request was closed
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
Bumps opentelemetry_sdk from 0.28.0 to 0.32.1.
Changelog
Sourced from opentelemetry_sdk's changelog.
... (truncated)
Commits
You can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.