getsentry / sentry

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

feat(alerts): Update eap alerts to display warning banner when low confidence data is detected #81105

Open edwardgou-sentry opened 19 hours ago

edwardgou-sentry commented 19 hours ago

Updates EAP Alerts and supporting timeseries utilities (eventsRequest, etc) to handle confidence if it is returned by Discover. This PR assumes that discover timeseries intervals will include a confidence attribute alongside the original count, ie:

[timestamp, [{count: ...., confidence: HIGH|LOW}]]

example:
[1731556800, [{count: 100, confidence: 'LOW'}]],
[1731560400, [{count: 200, confidence: 'HIGH'}]],

If any low confidence interval is detected in the timeseries response of an EAP Alert, we display a warning banner.

image
codecov[bot] commented 19 hours ago

Codecov Report

Attention: Patch coverage is 93.54839% with 2 lines in your changes missing coverage. Please review.

:white_check_mark: All tests successful. No failed tests found.

Files with missing lines Patch % Lines
static/app/views/alerts/rules/metric/ruleForm.tsx 33.33% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #81105 +/- ## ========================================== - Coverage 78.46% 78.46% -0.01% ========================================== Files 7209 7218 +9 Lines 319450 319805 +355 Branches 43959 44038 +79 ========================================== + Hits 250670 250927 +257 - Misses 62383 62481 +98 Partials 6397 6397 ```
codecov[bot] commented 6 hours ago

Bundle Report

Changes will increase total bundle size by 16.76kB (0.05%) :arrow_up:. This is within the configured threshold :white_check_mark:

Detailed changes | Bundle name | Size | Change | | ----------- | ---- | ------ | | app-webpack-bundle-array-push | 31.95MB | 16.76kB (0.05%) :arrow_up: |