Skip to content

Conversation

@dianed-square
Copy link
Contributor

@dianed-square dianed-square commented Oct 29, 2025

This PR documents OpenTelemetry Protocol (OTLP) configuration support. Also changed to lowercase “goose” in touched files.

Documentation updates:

  • documentation/docs/guides/environment-variables.md:
    • Add new “Observability” section and made Langfuse a subsection
    • Add “OpenTelemetry Protocol (OTLP)” subsection with OTEL_EXPORTER_OTLP_ENDPOINT and OTEL_EXPORTER_OTLP_TIMEOUT vars
  • documentation/docs/guides/config-file.md:
    • Add otel_exporter_otlp_endpoint and otel_exporter_otlp_timeout
  • documentation/docs/troubleshooting.md:
    • Add “Debugging and Diagnostics” section with links to logs and observability

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

Testing

Manual testing - Verified OTLP export via env vars (Jaeger in Docker received traces). I was unable to verify with config.yaml keys but goose says it should work as described


@dianed-square dianed-square requested a review from a team as a code owner October 29, 2025 06:47
@github-actions
Copy link
Contributor

github-actions bot commented Oct 29, 2025

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-10-29 17:37 UTC

@dianed-square dianed-square merged commit 565dcb8 into main Oct 29, 2025
15 checks passed
@dianed-square dianed-square deleted the docs/otlp branch October 29, 2025 17:36
fbalicchia pushed a commit to fbalicchia/goose that referenced this pull request Nov 7, 2025
BlairAllan pushed a commit to BlairAllan/goose that referenced this pull request Nov 29, 2025
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

Successfully merging this pull request may close these issues.

3 participants