-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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(beacon): Events in 24h for now calculated properly #62441
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #62441 +/- ##
==========================================
+ Coverage 81.21% 81.26% +0.04%
==========================================
Files 5188 5197 +9
Lines 228782 230095 +1313
Branches 38423 39764 +1341
==========================================
+ Hits 185812 186989 +1177
- Misses 37302 37376 +74
- Partials 5668 5730 +62
|
for organization_id in organization_ids: | ||
events_per_org_24h = tsdb.backend.get_sums( | ||
model=TSDBModel.organization_total_received, | ||
keys=[organization_id], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
multiple org ids cannot be passed in here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed a couple of small changes to make the test more readable offline, but otherwise LGTM.
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
events in last 24 hours were always showing as 0, this fixes that
events in last 24 hours were always showing as 0, this fixes that
events in last 24 hours were always showing as 0, this fixes that