Skip to content

chore: remove legacy OTEL forwarding - #5979

Merged
tgmendes merged 11 commits into
mainfrom
chore/remove-legacy-otel-forwarding
Sep 2, 2026
Merged

chore: remove legacy OTEL forwarding#5979
tgmendes merged 11 commits into
mainfrom
chore/remove-legacy-otel-forwarding

Conversation

@tgmendes

@tgmendes tgmendes commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove the raw OTLP body-tee middleware, forwarding worker, org-wide configuration service, and dashboard settings section.
  • Remove the retired management API, generated CLI/SDK surface, and audit writer.
  • Keep the project-scoped normalized OTLP relay introduced by feat: route normalized OTLP signals to project destinations #5939 as the only customer forwarding path.
  • Leave the unused legacy database table in place for a separate migration-only contract PR.

Motivation

#5939 routes normalized logs, metrics, and traces through active project data-export routes, with destination headers and sensitive-data policy applied after normalization. Retaining the raw org-wide forwarder would duplicate delivery and preserve a competing configuration source. This stacked cleanup makes the normalized OTLP pipeline and data-export routes authoritative.

This is a hard cutover: existing org-wide forwarding rows are not migrated and stop taking effect. Active legacy configurations must have equivalent project data-export routes before merge.

@tgmendes
tgmendes requested review from a team as code owners September 2, 2026 12:58
@tgmendes tgmendes added go Pull requests that update go code javascript Pull requests that update javascript code labels Sep 2, 2026
@changeset-bot

changeset-bot Bot commented Sep 2, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: caa7b17

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Comment thread server/internal/outbox/events/catalog_gen.yaml

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 57 files

Re-trigger cubic

@tgmendes

tgmendes commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

Demo

Logging settings after legacy forwarding removal

What it shows:

  1. The Logging & Telemetry page retains the organization telemetry controls.
  2. The legacy OTEL forwarding destination form is no longer present; data-export routes own destination configuration.

@tgmendes
tgmendes force-pushed the chore/remove-legacy-otel-forwarding branch from 4db6c55 to 8697e0b Compare September 2, 2026 13:05
Base automatically changed from feat/normalized-otel-data-export-routing to main September 2, 2026 14:05
@tgmendes
tgmendes added this pull request to the merge queue Sep 2, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Sep 2, 2026
@tgmendes
tgmendes added this pull request to the merge queue Sep 2, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Sep 2, 2026
# Conflicts:
#	server/internal/otel/signal_relay.go
@tgmendes
tgmendes added this pull request to the merge queue Sep 2, 2026
Merged via the queue into main with commit 83973ad Sep 2, 2026
44 checks passed
@tgmendes
tgmendes deleted the chore/remove-legacy-otel-forwarding branch September 2, 2026 14:35
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

go Pull requests that update go code javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants