-
Notifications
You must be signed in to change notification settings - Fork 612
Description
Related Problems?
As far as I can tell, the documentation for the opentelemetry_sdk crate doesn't say anywhere what environment variables it does or does not recognize. The only authoritative documentation appears to be this table, which is somewhat difficult to read given its width: https://github.com/open-telemetry/opentelemetry-specification/blob/main/spec-compliance-matrix.md#environment-variables
My team believed that the Rust SDK supports all the standard OTEL environment variables (possibly because we didn't notice the "note" on this page, but I don't recall specifically how we came to this conclusion). We therefore relied on OTEL_TRACES_SAMPLER to configure our sampling, and consequently emitted 100x more traces than intended.
Describe the solution you'd like:
The opentelemetry_sdk crate itself should clearly document which environment variables are respected, and which are not.
Considered Alternatives
- The crate could link to the compliance matrix somewhere prominent in the documentation. It seems odd for me that the canonical source for this information would be part of the specification repo, though.
- The crate could support all environment variables. I think this should be the goal eventually.
Additional Context
No response
Tip
React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.