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

Change default OpenTelemetry protocol to http/protobuf #41522

Open
brunobat opened this issue Jun 27, 2024 · 6 comments
Open

Change default OpenTelemetry protocol to http/protobuf #41522

brunobat opened this issue Jun 27, 2024 · 6 comments

Comments

@brunobat
Copy link
Contributor

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 #41521

Implementation ideas

No response

Copy link

quarkus-bot bot commented Jun 27, 2024

/cc @radcortez (opentelemetry)

@geoand
Copy link
Contributor

geoand commented Jul 2, 2024

So I assume we need #41521 in first?

@brunobat
Copy link
Contributor Author

brunobat commented Jul 2, 2024

Yes we do and #39032 as well, preferably.

@geoand
Copy link
Contributor

geoand commented Jul 2, 2024

👌

@jorsol
Copy link
Contributor

jorsol commented Oct 28, 2024

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?

@brunobat
Copy link
Contributor Author

brunobat commented Oct 28, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

No branches or pull requests

3 participants