Skip to content

fix: Set transaction source for unsampled streamed spans#6250

Closed
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/unsampled-source
Closed

fix: Set transaction source for unsampled streamed spans#6250
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/unsampled-source

Conversation

@alexander-alderman-webb
Copy link
Copy Markdown
Contributor

@alexander-alderman-webb alexander-alderman-webb commented May 11, 2026

Description

Set the transaction source that is attached to events before the early return for NoOpStreamedSpan.

Enables creating a streaming lifecycle version of tests/integrations/django/test_basic.py::test_transaction_style.

Issues

Reminders

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 11, 2026

Codecov Results 📊

146 passed | Total: 146 | Pass Rate: 100% | Execution Time: 21.43s

All tests are passing successfully.

❌ Patch coverage is 0.00%. Project has 14324 uncovered lines.

Files with missing lines (1)
File Patch % Lines
scope.py 64.06% ⚠️ 322 Missing and 89 partials

Generated by Codecov Action

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review May 11, 2026 07:59
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner May 11, 2026 07:59
@alexander-alderman-webb
Copy link
Copy Markdown
Contributor Author

I think the test is actually wrong, will update the test instead.

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.

1 participant