Rename otlp_exporter to otlp_grpc_exporter#2208
Merged
albertlockett merged 4 commits intoopen-telemetry:mainfrom Mar 9, 2026
Merged
Rename otlp_exporter to otlp_grpc_exporter#2208albertlockett merged 4 commits intoopen-telemetry:mainfrom
albertlockett merged 4 commits intoopen-telemetry:mainfrom
Conversation
3c6544e to
8b8125d
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2208 +/- ##
=======================================
Coverage 87.44% 87.44%
=======================================
Files 558 558
Lines 185764 185764
=======================================
+ Hits 162447 162450 +3
+ Misses 22791 22788 -3
Partials 526 526
🚀 New features to boost your workflow:
|
- Rename otlp_exporter.rs -> otlp_grpc_exporter.rs - Update URN: urn:otel:exporter:otlp -> urn:otel:exporter:otlp_grpc - Update lib.rs module declaration - Fix otlp_http_exporter imports referencing old module - Update all YAML configs, perf test templates, validation configs - Update documentation (ARCHITECTURE.md, self_tracing_architecture.md, metrics-guide.md, README.md) - Update node_urn.rs and pipeline.rs test references
8b8125d to
bad5226
Compare
drewrelmas
approved these changes
Mar 6, 2026
Contributor
drewrelmas
left a comment
There was a problem hiding this comment.
On the surface this looks good to me, will wait for @albertlockett to take a look as he requested the work.
Contributor
Author
|
thanks @drewrelmas waiting for @albertlockett feedback |
albertlockett
approved these changes
Mar 9, 2026
cijothomas
pushed a commit
to cijothomas/otel-arrow
that referenced
this pull request
Mar 11, 2026
## Summary Renames the gRPC-based OTLP exporter module and URN to distinguish it from the newly-added HTTP-based exporter (open-telemetry#2070). **URN change:** `urn:otel:otlp:exporter` → `urn:otel:otlp_grpc:exporter` **Module rename:** `otlp_exporter.rs` → `otlp_grpc_exporter.rs` Fixes open-telemetry#2107 ## Changes ### Rust Source (3 files) - **Renamed** `otlp_exporter.rs` → `otlp_grpc_exporter.rs` and updated URN constant value - **Updated** `lib.rs` module declaration: `pub mod otlp_exporter` → `pub mod otlp_grpc_exporter` - **Updated** `urn.rs` test case URN reference ### Config Files (8 files) All in `rust/otap-dataflow/configs/` — replaced `plugin_urn` from `urn:otel:otlp:exporter` → `urn:otel:otlp_grpc:exporter` ### Perf Test Templates (9 files) All in `tools/pipeline_perf_test/test_suites/integration/templates/configs/` — same URN replacement ### Documentation (3 files) - `crates/quiver/ARCHITECTURE.md` — updated node names + URN in config examples - `docs/self_tracing_architecture.md` — updated node names in config example - `docs/telemetry/metrics-guide.md` — updated metric set name ## What Was NOT Changed (by design) - **Test function names** (e.g. `otlp_exporter_connects_with_mtls`) — describe behavior, not module path - **Test file names** (`otlp_exporter_tls.rs`, `otlp_exporter_proxy_tls.rs`) — no module imports depend on them - **Telemetry crate** (`otlp_exporter_provider`, `configure_grpc_otlp_exporter`) — separate OTel SDK, not the pipeline exporter - **Constant/struct names** (`OTLP_EXPORTER_URN`, `OTLPExporter`) — kept per issue scope ## Verification - ✅ `cargo build --workspace` — passed - ✅ `cargo test --workspace` — all tests passed, zero failures - ✅ `grep -r "urn:otel:otlp:exporter"` — zero matches remain Co-authored-by: Drew Relmas <drewrelmas@gmail.com> Co-authored-by: albertlockett <a.lockett@f5.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Renames the gRPC-based OTLP exporter module and URN to distinguish it from the newly-added HTTP-based exporter (#2070).
URN change:
urn:otel:otlp:exporter→urn:otel:otlp_grpc:exporterModule rename:
otlp_exporter.rs→otlp_grpc_exporter.rsFixes #2107
Changes
Rust Source (3 files)
otlp_exporter.rs→otlp_grpc_exporter.rsand updated URN constant valuelib.rsmodule declaration:pub mod otlp_exporter→pub mod otlp_grpc_exporterurn.rstest case URN referenceConfig Files (8 files)
All in
rust/otap-dataflow/configs/— replacedplugin_urnfromurn:otel:otlp:exporter→urn:otel:otlp_grpc:exporterPerf Test Templates (9 files)
All in
tools/pipeline_perf_test/test_suites/integration/templates/configs/— same URN replacementDocumentation (3 files)
crates/quiver/ARCHITECTURE.md— updated node names + URN in config examplesdocs/self_tracing_architecture.md— updated node names in config exampledocs/telemetry/metrics-guide.md— updated metric set nameWhat Was NOT Changed (by design)
otlp_exporter_connects_with_mtls) — describe behavior, not module pathotlp_exporter_tls.rs,otlp_exporter_proxy_tls.rs) — no module imports depend on themotlp_exporter_provider,configure_grpc_otlp_exporter) — separate OTel SDK, not the pipeline exporterOTLP_EXPORTER_URN,OTLPExporter) — kept per issue scopeVerification
cargo build --workspace— passedcargo test --workspace— all tests passed, zero failuresgrep -r "urn:otel:otlp:exporter"— zero matches remain