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

Investigate Stock OTel Configuration Experience #73

Closed
bdarfler opened this issue Jul 19, 2021 · 2 comments
Closed

Investigate Stock OTel Configuration Experience #73

bdarfler opened this issue Jul 19, 2021 · 2 comments
Assignees

Comments

@bdarfler
Copy link
Contributor

What is the Stock OTel Configuration experience with cmd line args, env vars, appsettings.json? Do they conflict? Is there a precedence hierarchy?

The intention here is to have this investigation inform what if any changes we would like to make to the beestro.

@robbkidd
Copy link
Member

robbkidd commented Jul 26, 2021

For environment variable configuration of stock OTel dotnet, I've had to use a naming convention that .NET uses to override appsettings (example: export Otlp__Endpoint="https://api.honeycomb.io:443") instead of the standard OTLP exporter spec environment variables (example: export OTEL_EXPORTER_OTLP_ENDPOINT="https://api.honeycomb.io:443"). The only place I see the spec'd environment variable used in stock OTel dotnet is in the integration test where the test itself looks up the value of this environment variable to pass it to an OtlpTraceExporter constructor. Stock OTel libraries themselves don't appear to use the spec'd environment variables.

Update: Literally hours ago, a PR opened to work the open issue add support for the OTLP Exporter environment variables from the spec.

@robbkidd
Copy link
Member

And the PR merged with support for a few, but not all of the OTLP Exporter spec's environment variables.

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

3 participants