deploy-sourcegraph-docker: make Jaeger optional, always deploy otel-collector #40455
Assignees
Comments
13 tasks
This was referenced Sep 2, 2022
bobheadxi
added a commit
to sourcegraph/deploy-sourcegraph-docker
that referenced
this issue
Sep 6, 2022
#849 took a conservative approach to sourcegraph/sourcegraph#40455 by keeping Jaeger around. I've since reached out to the only customer using this deployment mechanism (https://sourcegraph.slack.com/archives/C012V43N8E7/p1662138872279299) which indicates there is no strong interest in keeping Jaeger around, so this change removes it entirely.
bobheadxi
added a commit
that referenced
this issue
Sep 7, 2022
Now that all our core deployment methods bundle OpenTelemetry Collector by default (#40456, #40457, #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 #39399
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: