Skip to content

Conversation

@kcons
Copy link
Member

@kcons kcons commented Oct 23, 2025

There are multiple ways requests to Seer can fail, and we'd like to track them independently.
This embeds the failure text in the logger call to group them separately, and also switches to
logger.warning because logger.error changes block deploy.

@kcons kcons requested a review from a team as a code owner October 23, 2025 00:37
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 23, 2025
@codecov
Copy link

codecov bot commented Oct 23, 2025

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
.../sentry/seer/anomaly_detection/get_anomaly_data.py 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #101982      +/-   ##
===========================================
+ Coverage   80.58%    80.97%   +0.39%     
===========================================
  Files        8729      8729              
  Lines      388387    388389       +2     
  Branches    24625     24625              
===========================================
+ Hits       312977    314505    +1528     
+ Misses      75050     73524    -1526     
  Partials      360       360              

@kcons kcons requested a review from mifu67 October 23, 2025 18:06
@mifu67 mifu67 merged commit d683693 into master Oct 23, 2025
70 checks passed
@mifu67 mifu67 deleted the kcons/refined branch October 23, 2025 18:41
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