[go: up one dir, main page]
Skip to content

distributed tracing: Migrate from OpenTracing to OpenTelemetry

Olivier Campeaurequested to merge
oc/use-open-telemetry into master

This commit migrates Gitaly from OpenTracing to OpenTelemetry for distributed tracing.

It is linked to this issue: #4329

At first, this MR was dependent on another MR in the LabKit project. The initial idea was to migrate LabKit to OpenTelemetry, and then use this new feature inside Gitaly.

But after discussing with @qmnguyen0711, we decided to drop LabKit and implement the tracer initialization within Gitaly itself.

It turns out that @qmnguyen0711 had already started working on migrating Gitaly to OpenTelemetry a couple months ago. See this MR.

So this MR is now a bridge between @qmnguyen0711's MR and my initial one. Some of the code here has been taken directly from his MR. The rest is mine.

This MR has been successfully tested with the following setup:

  1. Jaeger when running locally with GDK
  2. Google Stackdriver with a personal GCP account and API token

Reference:

Edited by Olivier Campeau

Merge request reports