Skip to content

[shipping] Migrate OTLP exporters from gRPC to http/protobuf#3619

Merged
julianocosta89 merged 2 commits into
open-telemetry:mainfrom
cijothomas:grpc-http/shipping
Jul 3, 2026
Merged

[shipping] Migrate OTLP exporters from gRPC to http/protobuf#3619
julianocosta89 merged 2 commits into
open-telemetry:mainfrom
cijothomas:grpc-http/shipping

Conversation

@cijothomas

@cijothomas cijothomas commented Jul 3, 2026

Copy link
Copy Markdown
Member

Changes

Migrates the shipping service's OTLP exporters (traces, metrics, logs) from gRPC to
http/protobuf, per #2676.

  • Swapped .with_tonic() for .with_http().with_protocol(Protocol::HttpBinary) in
    telemetry_conf.rs for all 3 exporters
  • Removed the now-unneeded grpc-tonic feature from Cargo.toml (HTTP support was already a
    default feature of opentelemetry-otlp 0.32, so no new dependencies needed)
  • Pointed shipping at the collector's HTTP port in compose.yaml

Verified locally: traces, native Rust metrics (http_server_*), and logs all confirmed flowing
after the switch.

Merge Requirements

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the docs
  • Appropriate Helm chart updates in the helm-charts

(After merge, will follow up with Helm PRs)

@github-actions github-actions Bot added the helm-update-required Requires an update to the Helm chart when released label Jul 3, 2026
@cijothomas cijothomas force-pushed the grpc-http/shipping branch from 794bd9b to f4ee707 Compare July 3, 2026 00:56
@cijothomas cijothomas marked this pull request as ready for review July 3, 2026 01:53
@cijothomas cijothomas requested a review from a team as a code owner July 3, 2026 01:53

@scottgerring scottgerring left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@cijothomas cijothomas force-pushed the grpc-http/shipping branch from f4ee707 to 42b5f03 Compare July 3, 2026 11:30
@julianocosta89 julianocosta89 enabled auto-merge July 3, 2026 12:15
@julianocosta89

Copy link
Copy Markdown
Member

@cijothomas same here

auto-merge was automatically disabled July 3, 2026 12:24

Head branch was pushed to by a user without write access

@cijothomas cijothomas force-pushed the grpc-http/shipping branch from 42b5f03 to 5155b52 Compare July 3, 2026 12:24
@cijothomas

Copy link
Copy Markdown
Member Author

@cijothomas same here

Could you re-queue to merge? My conflict resolution push kicked this off the merge queue.

@julianocosta89 julianocosta89 added this pull request to the merge queue Jul 3, 2026
Merged via the queue into open-telemetry:main with commit 5127e5d Jul 3, 2026
55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

helm-update-required Requires an update to the Helm chart when released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants