Skip to content

Inline OTLP metrics exporter#6465

Draft
shuheiktgw wants to merge 2 commits into
quickwit-oss:mainfrom
shuheiktgw:codex/inline-otlp-exporter
Draft

Inline OTLP metrics exporter#6465
shuheiktgw wants to merge 2 commits into
quickwit-oss:mainfrom
shuheiktgw:codex/inline-otlp-exporter

Conversation

@shuheiktgw
Copy link
Copy Markdown
Collaborator

Summary

  • Replace the metrics-exporter-otel dependency with a local OTLP metrics recorder implementation.
  • Keep the existing OTLP SDK version and protocol/temporality config behavior unchanged.
  • Move the metrics exporter tests into a dedicated module file.

Tests

  • make fmt
  • cargo test -p quickwit-telemetry-exporters --lib

@guilload
Copy link
Copy Markdown
Member

What's the rationale for doing this?

@shuheiktgw
Copy link
Copy Markdown
Collaborator Author

@guilload Right now, metrics-rs doesn’t support OTLP exporters by default. There is an open PR for it, but I’m not sure when it will be merged.

In the previous PR, I used metrics-exporter-otel instead, but it doesn’t seem to be very widely used, so I’m concerned that relying on it could introduce supply chain risk. There is another similar crate, but that also doesn’t seem to be widely adopted.

The exporter implementation is relatively simple, so I was thinking it might be better for us to just write it ourselves. What do you think?

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