Skip to content

tracing: enable opentelemetry by default, remove legacy values

Administrator requested to merge otel-default-tracing into main

Created by: bobheadxi

Now that all our deployment methods bundle OpenTelemetry Collector by default (https://github.com/sourcegraph/sourcegraph/issues/40456, https://github.com/sourcegraph/sourcegraph/issues/40457, https://github.com/sourcegraph/sourcegraph/issues/40455), we can now switch over to enabling OpenTelemetry by default. This change also:

  • removes the default traceURL template, since the Jaeger instance it points to by default no longer exists on most deployments.
  • fixes the behaviour of observability.tracing config, and adds testing
  • refactors some things in internal/trace to be OpenTelemetry-first

Closes https://github.com/sourcegraph/sourcegraph/issues/39399

Test plan

CI passes, manual testing

Merge request reports