Skip to content

feat(uptime): Add organization uptime summary endpoint #96710

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

Conversation

evanpurkhiser
Copy link
Member

@evanpurkhiser evanpurkhiser commented Jul 29, 2025

  • Add OrganizationUptimeSummaryEndpoint for returning some statistics about the uptime monitors. Right now it returns counts we can use to calculate uptime percentages.

  • Extract common utility function for subscription ID authorization and mapping

  • Support both EAP Item results and the legacy EAP checks table

@evanpurkhiser evanpurkhiser requested review from a team as code owners July 29, 2025 22:28
@evanpurkhiser evanpurkhiser removed the request for review from a team July 29, 2025 22:28
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 29, 2025
cursor[bot]

This comment was marked as outdated.

Copy link

codecov bot commented Jul 29, 2025

Codecov Report

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

Files with missing lines Patch % Lines
...ry/uptime/endpoints/organization_uptime_summary.py 96.66% 3 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           master   #96710    +/-   ##
========================================
  Coverage   80.64%   80.64%            
========================================
  Files        8471     8474     +3     
  Lines      373148   373275   +127     
  Branches    24208    24208            
========================================
+ Hits       300918   301039   +121     
- Misses      71895    71901     +6     
  Partials      335      335            

- Add OrganizationUptimePercentlesEndpoint for calculating uptime
  statistics. There might be a better name for this, but we already have
  `uptime-stats` and `uptime-counts`.

- Extract common utility function for subscription ID authorization and mapping

- Add UptimePercentileStats model and serializer for percentile calculations

- Support both EAP results and legacy uptime checks
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-uptime-add-organization-uptime-percentiles-endpoint branch from 86709e7 to 3a10fcc Compare July 29, 2025 22:40
@evanpurkhiser evanpurkhiser changed the title feat(uptime): Add organization uptime percentiles endpoint feat(uptime): Add organization uptime summary endpoint Jul 29, 2025
Copy link
Member

@JoshFerge JoshFerge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks reasonable to me.

@evanpurkhiser evanpurkhiser merged commit bc5ec7d into master Jul 30, 2025
69 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/feat-uptime-add-organization-uptime-percentiles-endpoint branch July 30, 2025 15:04
andrewshie-sentry pushed a commit that referenced this pull request Aug 4, 2025
- Add `OrganizationUptimeSummaryEndpoint` for returning some statistics
about the uptime monitors. Right now it returns counts we can use to
calculate uptime percentages.

- Extract common utility function for subscription ID authorization and
mapping

- Support both EAP Item results and the legacy EAP checks table
Copy link

sentry-io bot commented Aug 12, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

Did you find this useful? React with a 👍 or 👎

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.

2 participants