Skip to content

Add optional OpenTelemetry (OTEL) configuration for services#5270

Open
sghosh23 wants to merge 1 commit into
developfrom
otel-config
Open

Add optional OpenTelemetry (OTEL) configuration for services#5270
sghosh23 wants to merge 1 commit into
developfrom
otel-config

Conversation

@sghosh23

Copy link
Copy Markdown
Contributor

Summary

Adds optional OpenTelemetry (OTEL) configuration to the wire-server Helm chart.

  • New top-level otel config block in charts/wire-server/values.yaml:

    otel:
      enabled: false
      endpoint: http://otel-collector:4318
      protocol: http/protobuf
      insecure: true
  • When otel.enabled is true, the following env vars are injected into the service pods (brig, galley, gundeck, cargohold, spar, background-worker):

    • OTEL_EXPORTER_OTLP_ENDPOINT
    • OTEL_EXPORTER_OTLP_PROTOCOL
    • OTEL_EXPORTER_OTLP_INSECURE
    • OTEL_SERVICE_NAME (per service)
    • OTEL_TRACES_EXPORTER=otlp
  • Defaults to disabled, so behaviour is unchanged unless explicitly enabled.

Notes

  • No changelog entry added yet.

Introduce a top-level `otel` config block in the wire-server chart values.
When `otel.enabled` is true, OTEL_* environment variables are injected into
the brig, galley, gundeck, cargohold, spar, and background-worker pods.
@sghosh23 sghosh23 requested review from a team as code owners June 16, 2026 14:50
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants