sentry
sentry copied to clipboard
feat(uptime): Add api for creating new uptime monitors
This api is currently gated behind a feature flag, so that we can just allow this on orgs we want to test on. We shouldn't enable this on other orgs until we have billing related concerns sorted out.
Codecov Report
Attention: Patch coverage is 88.52459% with 7 lines in your changes missing coverage. Please review.
Project coverage is 78.24%. Comparing base (
ccbdb39) to head (e82ba27). Report is 30 commits behind head on master.
:white_check_mark: All tests successful. No failed tests found.
| Files | Patch % | Lines |
|---|---|---|
| src/sentry/uptime/endpoints/validators.py | 81.81% | 3 Missing and 1 partial :warning: |
| src/sentry/uptime/subscriptions/subscriptions.py | 66.66% | 1 Missing and 2 partials :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## master #75794 +/- ##
=======================================
Coverage 78.24% 78.24%
=======================================
Files 6818 6819 +1
Lines 303690 303749 +59
Branches 52236 52247 +11
=======================================
+ Hits 237612 237664 +52
- Misses 59687 59693 +6
- Partials 6391 6392 +1
| Files | Coverage Δ | |
|---|---|---|
| src/sentry/api/urls.py | 100.00% <100.00%> (ø) |
|
| src/sentry/features/temporary.py | 100.00% <100.00%> (ø) |
|
| src/sentry/uptime/detectors/tasks.py | 95.04% <100.00%> (-0.05%) |
:arrow_down: |
| ...try/uptime/endpoints/project_uptime_alert_index.py | 100.00% <100.00%> (ø) |
|
| src/sentry/uptime/subscriptions/subscriptions.py | 92.50% <66.66%> (-7.50%) |
:arrow_down: |
| src/sentry/uptime/endpoints/validators.py | 83.67% <81.81%> (-1.52%) |
:arrow_down: |
Suspect Issues
This pull request was deployed and Sentry observed the following issues:
- ‼️ KeyError: 'owner'
/api/0/projects/{organization_id_or_slug}/{proj...View Issue
Did you find this useful? React with a 👍 or 👎