Skip to content

When jaeger is disabled, don't set tracing ENVs

Gabriel Mazetto requested to merge brodock/distributed-tracing-fix into master

What does this Merge Request do?

We should not set GITLAB_TRACING or GITLAB_TRACING_URL in runit env variables when jaeger is disabled.

This prevents memory issues with the running applications, as they will be collecting traces but have nowhere to send them.

fix #859 (closed)

Merge Request checklist

  • Tests added for new functionality. If not, please raise Issue to follow-up.
  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Documentation added/updated, if needed.
  • gdk doctor test added, if needed.
Edited by Gabriel Mazetto

Merge request reports