Skip to content

feat(output): Support OTLP metrics#79

Draft
jsirianni wants to merge 1 commit intochore/supported-telemetry-typefrom
feat/otlp-output-metrics
Draft

feat(output): Support OTLP metrics#79
jsirianni wants to merge 1 commit intochore/supported-telemetry-typefrom
feat/otlp-output-metrics

Conversation

@jsirianni
Copy link
Copy Markdown
Member

Proposed Change

Checklist
  • Changes are tested
  • CI has passed

Comment on lines -299 to +329
"blitz.otlp_grpc.channel.size",
metric.WithDescription("Current size of the data channel"),
"blitz.otlp_grpc.log_channel.size",
metric.WithDescription("Current size of the log data channel"),
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is a breaking change, okay before v1 release.

@Dylan-M Dylan-M added the do not merge PR should not be merged in its current state label Apr 1, 2026
@Dylan-M
Copy link
Copy Markdown
Contributor

Dylan-M commented Apr 1, 2026

Deferring this PR in favor of a combined metrics/traces approach that builds on top of PR #78's telemetry package foundation. The combined stack will adopt the telemetry type system from this PR, extend it with traces support, and integrate metric/trace generators and output support in a single coordinated effort. See PIPE-785.

@Dylan-M Dylan-M self-assigned this Apr 10, 2026
Copy link
Copy Markdown
Contributor

@Dylan-M Dylan-M left a comment

Choose a reason for hiding this comment

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

These changes have been folded into a larger, more comprehensive effort per our Slack discussion. These PRs should be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge PR should not be merged in its current state

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants