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

Default to OTEL_EXPORTER_OTLP_PROTOCOL=grpc if ext-grpc and open-telemetry/transport-grpc available #1305

Closed
dkarlovi opened this issue May 6, 2024 · 2 comments

Comments

@dkarlovi
Copy link
Contributor

dkarlovi commented May 6, 2024

OTEL_EXPORTER_OTLP_PROTOCOL is said to have:

Default value: SDK-dependent

Is your feature request related to a problem?

Assuming both ext-grpc and open-telemetry/transport-grpc are installed, this shows intent to use gRPC.

One env var less to configure.

Describe the solution you'd like

The protocol could easily default to grpc in that case.

Describe alternatives you've considered

Stay as is.

@ChrisLightfootWild
Copy link
Contributor

The specification suggests that this should default to http/protobuf. Furthermore:

[1]: SDKs SHOULD default endpoint variables to use http scheme unless they have good reasons to choose https scheme for the default (e.g., for backward compatibility reasons in a stable SDK release).

So is this deemed a good enough reason to deviate? 🤔

--

Regarding the configuration of compression, I did note that:

[3]: If no compression value is explicitly specified, SIGs can default to the value they deem most useful among the supported options.

Whereas this was not explicitly suggested in relation to the protocol.

@dkarlovi
Copy link
Contributor Author

Ah, I didn't see that, for reference you mean

If no configuration is provided the default transport SHOULD be http/protobuf unless SDKs have good reasons to choose grpc as the default

In which case I agree it shouldn't deviate indeed. 👍 Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants