getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
36.96k stars 3.97k forks source link

feat(crons): Record broken monitor recovery analytic #69260

Closed davidenwang closed 1 week ago

davidenwang commented 1 week ago

Records the event from https://github.com/getsentry/sentry/pull/69259 when a monitor environment with a broken detection is resolved

fixes: https://github.com/getsentry/team-crons/issues/178

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.67%. Comparing base (8f21628) to head (989b214). Report is 2 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #69260 +/- ## ======================================= Coverage 79.67% 79.67% ======================================= Files 6444 6444 Lines 286416 286423 +7 Branches 49353 49355 +2 ======================================= + Hits 228201 228207 +6 - Misses 57778 57779 +1 Partials 437 437 ``` | [Files](https://app.codecov.io/gh/getsentry/sentry/pull/69260?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry) | Coverage Δ | | |---|---|---| | [src/sentry/monitors/logic/mark\_ok.py](https://app.codecov.io/gh/getsentry/sentry/pull/69260?src=pr&el=tree&filepath=src%2Fsentry%2Fmonitors%2Flogic%2Fmark_ok.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS9tb25pdG9ycy9sb2dpYy9tYXJrX29rLnB5) | `100.00% <100.00%> (ø)` | | ... and [6 files with indirect coverage changes](https://app.codecov.io/gh/getsentry/sentry/pull/69260/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry)