quarkus icon indicating copy to clipboard operation
quarkus copied to clipboard

Change default OpenTelemetry protocol to `http/protobuf`

Open brunobat opened this issue 1 year ago • 1 comments

Description

The default OTLP protocol must be changed from grpc to http/protobuf in order to align with the specification.

There is something wrong with the span exporter when we change the protocol and the fix is too complex for https://github.com/quarkusio/quarkus/pull/41521

Implementation ideas

No response

brunobat avatar Jun 27 '24 16:06 brunobat

/cc @radcortez (opentelemetry)

quarkus-bot[bot] avatar Jun 27 '24 16:06 quarkus-bot[bot]

So I assume we need https://github.com/quarkusio/quarkus/pull/41521 in first?

geoand avatar Jul 02 '24 10:07 geoand

Yes we do and https://github.com/quarkusio/quarkus/pull/39032 as well, preferably.

brunobat avatar Jul 02 '24 14:07 brunobat

👌

geoand avatar Jul 02 '24 14:07 geoand

Isn't gRPC generally more performant than HTTP/Protobuf?

Or is this change simply to adjust the default, which can still be explicitly configured to use gRPC if desired?

jorsol avatar Oct 28 '24 11:10 jorsol

It is to adjust with the upstream default, @jorsol. It is was moved to HTTP and we should align. gRPC will still be available and I don't plan to implement the change anytime soon.

brunobat avatar Oct 28 '24 15:10 brunobat