-
Notifications
You must be signed in to change notification settings - Fork 55
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
support standard OpenTelemetry envvars where appropriate #11
Comments
Supporting more Otel Auto Configurer environment variables would be great. Here are some environment variable that look particularly valuable when integrating the
|
With #35 merged I can finally start writing tests that validate envvar configs in a system test, so I'll be working on this in the next few days. |
@cyrille-leclerc iirc I chose OTEL_CLI_SERVICE_NAME because I wasn't aware of the otel specs for envvars at that point. I did some testing tricks in equinix-labs/otel-init-go#2 that I will port over here so we can not only add the envvars but test that they're working right. That can come after the initial changes to align with the otel specs though. |
Any updated on that? |
@spuliaiev-sfdc as of PR #120 things should be in good shape. I'm not aware of any envvars that don't match the specs. There are also now tests in place that check the vars all the way through otel-cli to the server. Please let me know if there's something important to you that's missing! |
Am I missing the change to read OTEL_SERVICE_NAME when OTEL_CLI_SERVICE_NAME is not provided? |
@spuliaiev-sfdc you're right, that needs fixed. I've started on #158. |
This should be resolved as of v0.3.0. There's more opportunity to use service attribute autodetection but that's a topic for antoher issue. Please open a new issue for envvars you need that aren't supported yet. |
via @ahayworth, otel-cli should support the standard envvars that make sense
https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/sdk-environment-variables.md
The text was updated successfully, but these errors were encountered: