-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add common language-agnostic environment variable configuration docs #1432
Add common language-agnostic environment variable configuration docs #1432
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Several problems in the example env vars.
content/en/docs/concepts/sdk-configuration/exporter-selection-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/exporter-selection-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/general-sdk-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/otlp-exporter-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/otlp-exporter-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/otlp-exporter-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/otlp-exporter-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/otlp-exporter-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/otlp-exporter-configuration.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Austin Parker <austin@ap2.io>
content/en/docs/concepts/sdk-configuration/exporter-selection-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/exporter-selection-configuration.md
Outdated
Show resolved
Hide resolved
content/en/docs/concepts/sdk-configuration/exporter-selection-configuration.md
Outdated
Show resolved
Hide resolved
…configuration.md Co-authored-by: Robert Pająk <pellared@hotmail.com>
this lgtm -- g2g @cartermp? |
Yep, I can always add more in future PRs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know it is merged but I have a few more comments
|
||
## Endpoint Configuration | ||
|
||
The following environment variables let you configure an OTLP/gRPC or OTLP/HTTP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Following env vars are not documented:
OTEL_EXPORTER_OTLP_PROTOCOL
OTEL_EXPORTER_OTLP_TRACES_PROTOCOL
OTEL_EXPORTER_OTLP_METRICS_PROTOCOL
|
||
**Example:** | ||
|
||
* gRPC: `export OTEL_EXPORTER_OTLP_ENDPOINT="my-api-endpoint:443"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* gRPC: `export OTEL_EXPORTER_OTLP_ENDPOINT="my-api-endpoint:443"` | |
* gRPC: `export OTEL_EXPORTER_OTLP_ENDPOINT="https://my-api-endpoint"` |
AFAIK all (or most) of the languages require http
or https
prefix do determine if secure on insecure gRPC connection should be used
**Example:** | ||
|
||
* gRPC: `export OTEL_EXPORTER_OTLP_ENDPOINT="my-api-endpoint:443"` | ||
* HTTP: `export OTEL_EXPORTER_OTLP_ENDPOINT="https://my-api-endpoint/"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be safer to set
* HTTP: `export OTEL_EXPORTER_OTLP_ENDPOINT="https://my-api-endpoint/"` | |
* HTTP: `export OTEL_EXPORTER_OTLP_ENDPOINT="https://my-api-endpoint"` |
|
||
### `OTEL_EXPORTER_OTLP_ENDPOINT` | ||
|
||
A base endpoint URL for any signal type, with an optionall-specified port |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A base endpoint URL for any signal type, with an optionall-specified port | |
A base endpoint URL for any signal type, with an optionally-specified port |
|
||
### `OTEL_EXPORTER_OTLP_TRACES_ENDPOINT` | ||
|
||
Endpoint URL for trace data only, with an optionall-specified port number. Must |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Endpoint URL for trace data only, with an optionall-specified port number. Must | |
Endpoint URL for trace data only, with an optionally-specified port number. Must |
|
||
### `OTEL_EXPORTER_OTLP_METRICS_ENDPOINT` | ||
|
||
Endpoint URL for trace data only, with an optionall-specified port number. Must |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Endpoint URL for trace data only, with an optionall-specified port number. Must | |
Endpoint URL for trace data only, with an optionally-specified port number. Must |
|
||
### `OTEL_EXPORTER_OTLP_LOGS_ENDPOINT` | ||
|
||
Endpoint URL for trace data only, with an optionall-specified port number. Must |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Endpoint URL for trace data only, with an optionall-specified port number. Must | |
Endpoint URL for trace data only, with an optionally-specified port number. Must |
Preview: https://deploy-preview-1432--opentelemetry.netlify.app/docs/concepts/sdk-configuration/
Okay, so I used the word "common" here, but this is mostly based on what I've seen people need to configure to successfully send data to a backend. There's a lot more stuff in the spec that could be pulled in here, but since perfect is the enemy of good, I figured I'd get some docs going that aren't just links to the spec.