Skip to content

Rip out opentracing tracing backend (take 2)#3896

Merged
DrJosh9000 merged 1 commit intov4from
kill-opentracing-take-2
May 5, 2026
Merged

Rip out opentracing tracing backend (take 2)#3896
DrJosh9000 merged 1 commit intov4from
kill-opentracing-take-2

Conversation

@DrJosh9000
Copy link
Copy Markdown
Contributor

Description

Repeat of #3867 but after rebasing v4 to incorporate changes from #3866 and #3880.

Original description

Opentracing was deprecated even when we added opentelemetry. We should get rid of it, but we haven't been able to make breaking changes. We're doing v4 now! Goodbye, opentracing 🫡

Changes

Remove the opentracing tracing backend. Only opentelemetry will be a permitted tracing backend now.

Testing

  • Tests have run locally (with go test ./...). Buildkite employees may check this if the pipeline has run automatically.
  • Code is formatted (with go tool gofumpt -extra -w .)

Disclosures / Credits

@moskyb gets credit for the previous PR, and that one said:

This one was mostly Amp.

@DrJosh9000 DrJosh9000 requested a review from moskyb May 5, 2026 23:46
@DrJosh9000 DrJosh9000 requested review from a team as code owners May 5, 2026 23:46
@DrJosh9000 DrJosh9000 changed the title Rip out opentracing tracing backend Rip out opentracing tracing backend (take 2) May 5, 2026
@DrJosh9000 DrJosh9000 merged commit 638c3b4 into v4 May 5, 2026
2 of 3 checks passed
@DrJosh9000 DrJosh9000 deleted the kill-opentracing-take-2 branch May 5, 2026 23:48
DrJosh9000 added a commit that referenced this pull request May 5, 2026
Rip out opentracing tracing backend (take 2)
DrJosh9000 added a commit that referenced this pull request May 6, 2026
Rip out opentracing tracing backend (take 2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants