feat(huey): Migrate Huey integration to spans-first tracing#6399
Merged
Conversation
The Huey integration was not properly handling task groups and chords when enqueuing. When a group/chord is enqueued, we would attempt to access the `name` attribute of the group/chord object for the span. They don't have one, causing an AttributeError. Fixes PY-2426 Fixes #6310
Groups and chords weren't introduced until Huey 3.0, so handle that gracefully.
…s set correctly on segments in _capture_exception method
ericapisani
changed the base branch from
master
to
py-2426-fix-group-and-chord-handling
Contributor
Codecov Results 📊✅ 282 passed | Total: 282 | Pass Rate: 100% | Execution Time: 43.69s All tests are passing successfully. ❌ Patch coverage is 17.65%. Project has 14882 uncovered lines. Files with missing lines (2)
Generated by Codecov Action |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Member
Author
|
bugbot run |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![sentry-warden[bot]](https://avatars.githubusercontent.com/in/2755838?s=60&v=4){kind=small}
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
…to test order of spans generated since that's stable
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 97fb86c. Configure here.
…t huey has in earlier versions that results in more events being emitted than a later version)
sentrivana
approved these changes
May 27, 2026
Contributor
sentrivana
left a comment
sentrivana
left a comment
There was a problem hiding this comment.
Nice 🚀 Two things: one bug in the old version of tests that's easy to fix, and the control flow exceptions stuff can now use the new mechanism from #6425
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Builds on a bug fix (PY-2426/ #6392 )
Migrates the Huey integration to support the new spans-first tracing lifecycle (
trace_lifecycle: "stream"), while keeping full backwards compatibility with the existing transaction-based path.Fixes PY-2331 and #6029