You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I can see that this variable is used in WithExportConfig which is implemented for TonicExporterBuilder and HttpExporterBuilder but this seems back to front since its value should presumably determine which of these is used? As far as I can see, the value isn't used after being set. For example, this unit test seems to be setting a tonic exporter to use http/protobuf.
Another thing in that example, was that I had to also check OTEL_EXPORTER_OTLP_ENDPOINT when using tonic in order to decide whether it needed to be configured with TLS or not.
It would be nice if these things were handled out of the box, maybe at the level of OtlpTracePipeline? Or maybe it is already possible and I've missed something 🙂.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
I can see that this variable is used in
WithExportConfig
which is implemented forTonicExporterBuilder
andHttpExporterBuilder
but this seems back to front since its value should presumably determine which of these is used? As far as I can see, the value isn't used after being set. For example, this unit test seems to be setting atonic
exporter to usehttp/protobuf
.When putting together an example recently that was configured through the environment, I checked the variable myself before deciding which kind of exporter to create and configure.
Another thing in that example, was that I had to also check
OTEL_EXPORTER_OTLP_ENDPOINT
when usingtonic
in order to decide whether it needed to be configured with TLS or not.It would be nice if these things were handled out of the box, maybe at the level of
OtlpTracePipeline
? Or maybe it is already possible and I've missed something 🙂.Beta Was this translation helpful? Give feedback.
All reactions