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

chore(rules): Add metrics on Redis operations #71143

Merged
merged 1 commit into from
May 20, 2024

Conversation

ceorourke
Copy link
Member

Add Datadog metrics when we perform a Redis operation.

@ceorourke ceorourke requested a review from a team May 17, 2024 23:16
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 17, 2024
Copy link

codecov bot commented May 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.91%. Comparing base (b0d328d) to head (235ef69).
Report is 12 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #71143      +/-   ##
==========================================
- Coverage   77.91%   77.91%   -0.01%     
==========================================
  Files        6525     6526       +1     
  Lines      290661   290663       +2     
  Branches    50294    50294              
==========================================
- Hits       226466   226463       -3     
- Misses      57947    57951       +4     
- Partials     6248     6249       +1     
Files Coverage Δ
src/sentry/buffer/redis.py 88.23% <100.00%> (+0.07%) ⬆️

... and 29 files with indirect coverage changes

@ceorourke ceorourke merged commit 9f3d24d into master May 20, 2024
50 checks passed
@ceorourke ceorourke deleted the ceorourke/add-metrics-redis-operations branch May 20, 2024 16:18
cmanallen pushed a commit that referenced this pull request May 21, 2024
Add Datadog metrics when we perform a Redis operation.
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.

None yet

2 participants