Skip to content

fix: Add root transaction to standalone v1 spans#6004

Merged
elramen merged 6 commits into
masterfrom
elramen-ds
May 19, 2026
Merged

fix: Add root transaction to standalone v1 spans#6004
elramen merged 6 commits into
masterfrom
elramen-ds

Conversation

@elramen
Copy link
Copy Markdown
Member

@elramen elramen commented May 19, 2026

Add root transaction/segment name and trace id as sentry.dsc.x attributes to all v1 spans. Needed for the new dynamic sampling pipeline to work.

Fixes https://linear.app/getsentry/issue/RELAY-246/add-root-transactionsegment-name-to-v1-spans
Ref https://linear.app/getsentry/issue/RELAY-239/add-dsc-attributes-trace-root-project-onto-every-span

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 19, 2026

RELAY-246

RELAY-239

@elramen elramen marked this pull request as ready for review May 19, 2026 09:32
@elramen elramen requested a review from a team as a code owner May 19, 2026 09:32
Comment thread relay-server/src/processing/legacy_spans/store.rs Outdated
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a3f9871. Configure here.

Comment thread relay-server/src/processing/legacy_spans/mod.rs Outdated
@elramen elramen requested a review from Dav1dde May 19, 2026 11:39
Comment thread relay-event-normalization/src/eap/mod.rs
Comment thread CHANGELOG.md Outdated
@elramen elramen enabled auto-merge May 19, 2026 13:13
@elramen elramen added this pull request to the merge queue May 19, 2026
Merged via the queue into master with commit 969c0e1 May 19, 2026
33 checks passed
@elramen elramen deleted the elramen-ds branch May 19, 2026 13:53
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