open-telemetry / opentelemetry-dotnet

The OpenTelemetry .NET Client
https://opentelemetry.io
Apache License 2.0
3.18k stars 753 forks source link

Add an optional gRPC client-side load balancing for the otel exporter #3272

Open faustodavid opened 2 years ago

faustodavid commented 2 years ago

Feature Request

What do you want to happen instead? What is the expected behavior?

Have an optional gRPC client-side load balancing for the otel exporter. https://docs.microsoft.com/en-us/aspnet/core/grpc/loadbalancing?view=aspnetcore-6.0

Describe alternatives you've considered.

Which alternative solutions or features have you considered?

Use the OtlpExportProtocol.HttpProtobuf, but then we won't have the performance advantages of using GRPC.

faustodavid commented 2 years ago

I was thinking maybe we could add a flag in the OtlpExporterOptions and use it to enable the client-side load balancing here:

https://github.com/open-telemetry/opentelemetry-dotnet/blob/e7f06afd51489e3047f167d9b7362aba512508c0/src/OpenTelemetry.Exporter.OpenTelemetryProtocol/OtlpExporterOptionsExtensions.cs#L45

github-actions[bot] commented 4 days ago

This issue was marked stale due to lack of activity and will be closed in 7 days. Commenting will instruct the bot to automatically remove the label. This bot runs once per day.