Skip to content

feat(celery): Support span streaming#6074

Draft
sentrivana wants to merge 5 commits intomasterfrom
ivana/migrate-celery-to-span-first
Draft

feat(celery): Support span streaming#6074
sentrivana wants to merge 5 commits intomasterfrom
ivana/migrate-celery-to-span-first

Conversation

@sentrivana
Copy link
Copy Markdown
Contributor

@sentrivana sentrivana commented Apr 14, 2026

Description

Support streaming mode in the Celery integration.

This is the first integration that contains explicit guards against creating certain spans as segments, mirroring how start_span works currently -- it won't create a span if there's no transaction. In Celery, there were multiple spots that, if we just replaced all legacy start_spans with the new traces.start_span, would've created new segment spans, leading to a change in behavior.

TODO:

  • replace APIs
  • map data to attributes from conventions
  • run tests in streaming mode

Issues

@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 14, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 14, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 14, 2026

Codecov Results 📊

142 passed | Total: 142 | Pass Rate: 100% | Execution Time: 22.00s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +116
Passed Tests 📈 +116
Failed Tests
Skipped Tests

All tests are passing successfully.

❌ Patch coverage is 0.00%. Project has 14186 uncovered lines.
✅ Project coverage is 33.71%. Comparing base (base) to head (head).

Files with missing lines (1)
File Patch % Lines
__init__.py 3.26% ⚠️ 267 Missing
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    27.64%    33.71%    +6.07%
==========================================
  Files          190       190         —
  Lines        21358     21399       +41
  Branches      7066      7096       +30
==========================================
+ Hits          5903      7213     +1310
- Misses       15455     14186     -1269
- Partials       561       700      +139

Generated by Codecov Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate celery to span first

1 participant