Skip to content

Align OpenTelemetry crates for opentelemetry_sdk 0.32.1#1717

Merged
nikhilsinhaparseable merged 2 commits into
parseablehq:mainfrom
ygndotgg:review-pr-1706
Jul 6, 2026
Merged

Align OpenTelemetry crates for opentelemetry_sdk 0.32.1#1717
nikhilsinhaparseable merged 2 commits into
parseablehq:mainfrom
ygndotgg:review-pr-1706

Conversation

@ygndotgg

@ygndotgg ygndotgg commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Fixes #XXXX.

Description

This PR updates the OpenTelemetry dependency stack to a compatible 0.32.x/0.33.x set, resolving the build failure caused by
mixed opentelemetry versions in the dependency graph.

The original bump to opentelemetry_sdk 0.32.1 introduced trait mismatches between opentelemetry_sdk, opentelemetry-otlp,
opentelemetry-stdout, opentelemetry-proto, and tracing-opentelemetry. To fix this, the telemetry dependencies were aligned
to matching versions so the tracing/export pipeline compiles cleanly again.

Key changes

  • Updated opentelemetry to 0.32.0
  • Updated opentelemetry_sdk to 0.32.1
  • Updated opentelemetry-otlp to 0.32.0
  • Updated opentelemetry-stdout to 0.32.0
  • Updated opentelemetry-proto to 0.32.0
  • Updated tracing-opentelemetry to 0.33.0
  • Refreshed Cargo.lock to reflect the resolved dependency graph

Summary by CodeRabbit

  • Chores
    • Updated OpenTelemetry-related dependencies to newer versions.
    • Kept existing telemetry features unchanged while aligning to the latest compatible releases.

@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: dbec4146-b328-46ff-b288-317f94b11d30

📥 Commits

Reviewing files that changed from the base of the PR and between 91632dd and ddda05b.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • Cargo.toml
🚧 Files skipped from review as they are similar to previous changes (1)
  • Cargo.toml

Walkthrough

This PR updates OpenTelemetry-related dependency versions in Cargo.toml while keeping the existing feature configuration unchanged.

Changes

Telemetry dependency upgrade

Layer / File(s) Summary
Update telemetry dependency versions
Cargo.toml
Bumped tracing-opentelemetry, opentelemetry, opentelemetry_sdk, opentelemetry-otlp, and opentelemetry-stdout to newer releases, preserving the current feature lists.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Poem

Hop, hop, the versions glide,
Tiny crates on a newer ride.
Features held, no fluff, no fuss,
Telemetry now hops with us 🐰

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the OpenTelemetry dependency alignment and matches the main change.
Description check ✅ Passed The description includes the goal, rationale, and key changes, but leaves the issue reference as a placeholder.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
Cargo.toml (1)

121-129: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Remove the unused rt-tokio featureopentelemetry_sdk only feeds BatchSpanProcessor here, so keep rt-tokio only if another workspace path needs Tokio runtime integration.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@Cargo.toml` around lines 121 - 129, The opentelemetry_sdk dependency is
enabling the unused rt-tokio feature even though this crate only uses
BatchSpanProcessor here; remove the rt-tokio feature from the opentelemetry_sdk
entry unless another symbol in the workspace explicitly needs Tokio runtime
integration. Keep the dependency versions unchanged and update the Cargo.toml
dependency declaration accordingly.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@Cargo.toml`:
- Around line 121-129: The opentelemetry_sdk dependency is enabling the unused
rt-tokio feature even though this crate only uses BatchSpanProcessor here;
remove the rt-tokio feature from the opentelemetry_sdk entry unless another
symbol in the workspace explicitly needs Tokio runtime integration. Keep the
dependency versions unchanged and update the Cargo.toml dependency declaration
accordingly.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 763c617e-6b1d-4724-ae45-b4a78c586cd6

📥 Commits

Reviewing files that changed from the base of the PR and between 80f1bd6 and 91632dd.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • Cargo.toml

dependabot Bot and others added 2 commits July 6, 2026 10:00
Bumps [opentelemetry_sdk](https://github.com/open-telemetry/opentelemetry-rust) from 0.31.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](open-telemetry/opentelemetry-rust@v0.31.0...opentelemetry-semantic-conventions-0.32.1)

---
updated-dependencies:
- dependency-name: opentelemetry_sdk
  dependency-version: 0.32.1
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@nikhilsinhaparseable nikhilsinhaparseable merged commit d0390ce into parseablehq:main Jul 6, 2026
12 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.

3 participants