Skip to content

ref: add metric when canonical key falls back #70385

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

Merged
merged 1 commit into from
May 7, 2024

Conversation

asottile-sentry
Copy link
Member

want to make sure this isn't happening before I clean up all of this -- I'm told this is handled by relay now

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 6, 2024
Copy link

codecov bot commented May 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.97%. Comparing base (fd8d3df) to head (49edd29).
Report is 13 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #70385      +/-   ##
==========================================
- Coverage   79.97%   79.97%   -0.01%     
==========================================
  Files        6495     6495              
  Lines      289790   289817      +27     
  Branches    49928    49933       +5     
==========================================
+ Hits       231753   231773      +20     
- Misses      57625    57632       +7     
  Partials      412      412              
Files Coverage Δ
src/sentry/utils/canonical.py 97.01% <100.00%> (+0.24%) ⬆️

... and 9 files with indirect coverage changes

@asottile-sentry asottile-sentry merged commit 571c43c into master May 7, 2024
50 checks passed
@asottile-sentry asottile-sentry deleted the asottile-canonical-key-obsolete branch May 7, 2024 16:48
@asottile-sentry
Copy link
Member Author

unfortunately it looks like this is still used according to the metrics so backing this out

@asottile-sentry asottile-sentry added the Trigger: Revert Add to a merged PR to revert it (skips CI) label May 7, 2024
@getsentry-bot
Copy link
Contributor

PR reverted: 21a2f1c

getsentry-bot added a commit that referenced this pull request May 7, 2024
This reverts commit 571c43c.

Co-authored-by: asottile-sentry <103459774+asottile-sentry@users.noreply.github.com>
@github-actions github-actions bot locked and limited conversation to collaborators May 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components Trigger: Revert Add to a merged PR to revert it (skips CI)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants