Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider supporting pass-through for OTLP metrics #515

Open
cartermp opened this issue Sep 16, 2022 · 5 comments
Open

Consider supporting pass-through for OTLP metrics #515

cartermp opened this issue Sep 16, 2022 · 5 comments
Labels
type: discussion Requests for comments, discussions about possible enhancements. type: enhancement New feature or request

Comments

@cartermp
Copy link
Member

For people who use refinery to sample traces and also send along metrics to honeycomb, it can be surprising that they can't "just point their metrics at refinery" and have them pass through. I've not seen anyone expect that the metrics should be sampled in some way, but having to set your endpoint differently for metrics can be surprising.

@cartermp cartermp added the type: discussion Requests for comments, discussions about possible enhancements. label Sep 16, 2022
@kentquirk
Copy link
Contributor

Yes -- this is already on the roadmap for sometime "soon" -- meaning we plan to do it but don't have a time commitment yet.

@cartermp cartermp added the type: enhancement New feature or request label Sep 19, 2022
@neoeinstein
Copy link

Dupe of #449?

@rasharab
Copy link

rasharab commented Jul 18, 2023

It seems like this is still a problem?

SEVERE: Failed to export metrics. Server responded with UNIMPLEMENTED. This usually means that your collector is not configured with an otlp receiver in the "pipelines" section of the configuration. If export is not desired and you are using OpenTelemetry autoconfiguration or the javaagent, disable export by setting OTEL_METRICS_EXPORTER=none. Full error message: unknown service opentelemetry.proto.collector.metrics.v1.MetricsService

@cartermp
Copy link
Member Author

@rasharab are you using HTTP or gRPC? For HTTP it should already be supported, but not for gRPC.

@rasharab
Copy link

Grpc. Thank you for the heads up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: discussion Requests for comments, discussions about possible enhancements. type: enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants