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

Add gRPC support for OTLP in addition to HTTP #5040

Closed
gayanper opened this issue May 5, 2024 · 3 comments
Closed

Add gRPC support for OTLP in addition to HTTP #5040

gayanper opened this issue May 5, 2024 · 3 comments

Comments

@gayanper
Copy link

gayanper commented May 5, 2024

Please describe the feature request.
Add support for grpc transport protocol for OTLP implementation for metrics.

Rationale
Some of the tools such as Microsoft .NET Aspire Dashboard which can be used as a developer testing tool to run locally for verification of metrics and traces can only be used if all three traces, metrics and logs are transported using grpc protocol. So it would be nice to implement the support for transporting metrics using grpc in addition to http as well.

Additional context
https://learn.microsoft.com/en-us/dotnet/aspire/fundamentals/dashboard/overview

@jonatan-ivanov
Copy link
Member

As far as I remember (according to the specs and the OTel maintainers), collector implementation must support all three transports (http/json, http/protobuf, grpc) so that they can talk to any clients. It's not the client (Micrometer) who should support all of them but the server. For example the OTel SDK only supports two out of three. I would try to open an issue to the Aspire Dashboard to support all the transport protocols.

@jonatan-ivanov jonatan-ivanov added waiting for feedback We need additional information before we can continue and removed waiting-for-triage labels May 6, 2024
@jonatan-ivanov jonatan-ivanov changed the title Add support for graph OTLP protocol in addition to http Add gRPC support for OTLP in addition to HTTP May 6, 2024
Copy link

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Copy link

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale May 21, 2024
@jonatan-ivanov jonatan-ivanov removed waiting for feedback We need additional information before we can continue feedback-reminder labels May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants