docs: update OTel environment variable and config guides#7221
docs: update OTel environment variable and config guides#7221codefromthecrypt merged 6 commits intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates OpenTelemetry documentation to describe enhanced environment variable support introduced in PR #7144. However, PR #7144 has not been merged yet, so this documentation describes features that do not currently exist in the codebase.
Changes:
- Updated
environment-variables.mdto document per-signal OTel exporter configuration (OTEL_TRACES_EXPORTER, OTEL_METRICS_EXPORTER, OTEL_LOGS_EXPORTER, OTEL_SDK_DISABLED) - Added comment to
config-files.mdrecommending environment variables over config file for OTel settings
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| documentation/docs/guides/environment-variables.md | Expands OpenTelemetry section with per-signal exporter configuration, signal-specific endpoints, and updated examples |
| documentation/docs/guides/config-files.md | Adds comment recommending ENV over config file for OTel configuration |
|
06df55b to
9e9545d
Compare
Signed-off-by: Adrian Cole <adrian@tetrate.io>
Signed-off-by: Adrian Cole <adrian@tetrate.io>
9e9545d to
f537c70
Compare
| # Debug traces to console (no collector needed) | ||
| export OTEL_TRACES_EXPORTER="console" | ||
|
|
||
| # Sample 10% of traces (reduce volume in production) |
There was a problem hiding this comment.
example for setting 10pct sampling as default is 100pct. 100pct is what a lot of APM backends want, as they can derive metrics from spans, but is expensive.
|
awesome! thanks..once your other PR is released..I think it will go out in the next release..then we can merge this! cc-ing: @dianed-square as well so she's aware. |
There was a problem hiding this comment.
what does this mean? excuse my ignorance.
There was a problem hiding this comment.
i think i tmeans you can put it in the config, as here, but if you have environment variables, it will use them instead?
There was a problem hiding this comment.
ohhh okay. I will suggest a slight rewrite for clarity. thanks mic
- Remove OTEL entries from Global Settings table - Remove OTEL block with awkward comment from Example Configuration - Add new Observability Configuration section with: - Proper hyperlink to environment variables guide - Table with lowercase keys (matching what code expects) - YAML example
|
|
||
| ## Observability Configuration | ||
|
|
||
| Configure goose to export telemetry to [OpenTelemetry](https://opentelemetry.io/docs/) compatible platforms. Environment variables override these settings and support additional options like per-signal configuration. See the [environment variables guide](/docs/guides/environment-variables#opentelemetry-protocol-otlp) for details. |
There was a problem hiding this comment.
The anchor link #opentelemetry-protocol-otlp is broken. The section in environment-variables.md was renamed from "OpenTelemetry Protocol (OTLP)" to "Observability Configuration", so the anchor should be #observability-configuration.
| Configure goose to export telemetry to [OpenTelemetry](https://opentelemetry.io/docs/) compatible platforms. Environment variables override these settings and support additional options like per-signal configuration. See the [environment variables guide](/docs/guides/environment-variables#opentelemetry-protocol-otlp) for details. | |
| Configure goose to export telemetry to [OpenTelemetry](https://opentelemetry.io/docs/) compatible platforms. Environment variables override these settings and support additional options like per-signal configuration. See the [environment variables guide](/docs/guides/environment-variables#observability-configuration) for details. |
* origin/main: docs: remove ALPHA_FEATURES flag from documentation (#7315) docs: escape variable syntax in recipes (#7314) docs: update OTel environment variable and config guides (#7221) docs: system proxy settings (#7311) docs: add Summon extension tutorial and update Skills references (#7310) docs: agent session id (#7289) fix(gemini-cli): restore streaming lost in #7247 (#7291) Update more instructions (#7305) feat: add Moonshot and Kimi Code declarative providers (#7304) fix(cli): handle Reasoning content and fix streaming thinking display (#7296) feat: add GOOSE_SUBAGENT_MODEL and GOOSE_SUBAGENT_PROVIDER config options (#7277) fix(openai): support "reasoning" field alias in streaming deltas (#7294) fix(ui): revert app-driven iframe width and send containerDimensions per ext-apps spec (#7300) New OpenAI event (#7301) ci: add fork guards to scheduled workflows (#7292)
* main: (54 commits) docs: add monitoring subagent activity section (#7323) docs: document Desktop UI recipe editing for model/provider and extensions (#7327) docs: add CLAUDE_THINKING_BUDGET and CLAUDE_THINKING_ENABLED environm… (#7330) fix: display 'Code Mode' instead of 'code_execution' in CLI (#7321) docs: add Permission Policy documentation for MCP Apps (#7325) update RPI plan prompt (#7326) docs: add CLI syntax highlighting theme customization (#7324) fix(cli): replace shell-based update with native Rust implementation (#7148) docs: rename Code Execution extension to Code Mode extension (#7316) docs: remove ALPHA_FEATURES flag from documentation (#7315) docs: escape variable syntax in recipes (#7314) docs: update OTel environment variable and config guides (#7221) docs: system proxy settings (#7311) docs: add Summon extension tutorial and update Skills references (#7310) docs: agent session id (#7289) fix(gemini-cli): restore streaming lost in #7247 (#7291) Update more instructions (#7305) feat: add Moonshot and Kimi Code declarative providers (#7304) fix(cli): handle Reasoning content and fix streaming thinking display (#7296) feat: add GOOSE_SUBAGENT_MODEL and GOOSE_SUBAGENT_PROVIDER config options (#7277) ...
Summary
Updates documentation to canonical otel ENV
Type of Change
AI Assistance
Related Issues
Docs from #7144