Skip to content

Conversation

@roaga
Copy link
Member

@roaga roaga commented Nov 15, 2025

Before we always selected the main thread, but often the main application logic is running in a different thread (e.g. in Seer, it's an AnyIO worker thread). Based on how the frontend determines which thread to show by default, the solution is to just use the thread from the first sample.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 15, 2025
@codecov
Copy link

codecov bot commented Nov 15, 2025

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/seer/explorer/utils.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #103426      +/-   ##
===========================================
- Coverage   80.78%    80.67%   -0.12%     
===========================================
  Files        9248      9243       -5     
  Lines      397172    394931    -2241     
  Branches    25158     25158              
===========================================
- Hits       320841    318594    -2247     
- Misses      75884     75890       +6     
  Partials      447       447              

@roaga roaga marked this pull request as ready for review November 17, 2025 15:54
@roaga roaga requested a review from a team as a code owner November 17, 2025 15:54
@roaga roaga merged commit 00917df into master Nov 17, 2025
67 checks passed
@roaga roaga deleted the explorer/fix-profile-thread-selection branch November 17, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants