Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: handle missing profile data in process_profile #87825

Merged
merged 2 commits into from
Mar 25, 2025

Conversation

sentry-autofix[bot]
Copy link
Contributor

👋 Hi there! This PR was automatically generated by Autofix 🤖

This fix was triggered by Rohan Agarwal

Fixes SENTRY-3QTA

  • Added checks to ensure profile, frames, stacks, and samples exist before processing.
  • Returns an empty list if any of these are missing to prevent errors.

If you have any questions or feedback for the Sentry team about this fix, please email [email protected] with the Run ID: 13395.

@roaga roaga requested a review from jennmueng March 25, 2025 03:46
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 25, 2025
Copy link

codecov bot commented Mar 25, 2025

Codecov Report

Attention: Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/seer/autofix.py 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #87825      +/-   ##
==========================================
- Coverage   87.75%   87.75%   -0.01%     
==========================================
  Files        9903     9903              
  Lines      562481   562485       +4     
  Branches    22144    22144              
==========================================
- Hits       493601   493599       -2     
- Misses      68450    68456       +6     
  Partials      430      430              

@roaga roaga added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 25, 2025
@roaga roaga marked this pull request as ready for review March 25, 2025 15:58
@roaga roaga requested a review from a team as a code owner March 25, 2025 15:58
@roaga roaga merged commit f868489 into master Mar 25, 2025
49 of 50 checks passed
@roaga roaga deleted the autofix/seer-autofix-missing-profile-data branch March 25, 2025 17:14
andrewshie-sentry pushed a commit that referenced this pull request Mar 27, 2025
👋 Hi there! This PR was automatically generated by Autofix 🤖

This fix was triggered by Rohan Agarwal

Fixes
[SENTRY-3QTA](https://sentry.io/organizations/sentry/issues/6454821631/)

- Added checks to ensure `profile`, `frames`, `stacks`, and `samples`
exist before processing.
- Returns an empty list if any of these are missing to prevent errors.

If you have any questions or feedback for the Sentry team about this
fix, please email [[email protected]](mailto:[email protected]) with the
Run ID: 13395.

---------

Co-authored-by: sentry-autofix[bot] <157164994+sentry-autofix[bot]@users.noreply.github.com>
Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
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 Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant