-
Notifications
You must be signed in to change notification settings - Fork 66
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: The disabled failed probe is not retrying when it encounters an incident after enabling it #1242
Fix: The disabled failed probe is not retrying when it encounters an incident after enabling it #1242
Conversation
…ix/not-retrying-enabled-probe
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1242 +/- ##
==========================================
+ Coverage 63.49% 63.58% +0.09%
==========================================
Files 111 111
Lines 3304 3304
Branches 569 567 -2
==========================================
+ Hits 2098 2101 +3
+ Misses 1016 1015 -1
+ Partials 190 188 -2 ☔ View full report in Codecov by Sentry. |
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.
LGTM
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.
Nicely done.
Monika Pull Request (PR)
What feature/issue does this PR add
The disabled failed probe is not retrying when it encounters an incident after enabling it.
How did you implement / how did you fix it
Remove incident data when disabling a probe.
How to test
Video
Before (Incident is not reported)
Before.mp4
After (Incident reported)
Remove.incident.state.mp4