-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Security Solution] Fix incorrect number of alerts shows under External alert trend #111746
[Security Solution] Fix incorrect number of alerts shows under External alert trend #111746
Conversation
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
7d9b4b9
to
db66aca
Compare
💚 Build SucceededMetrics [docs]Public APIs missing comments
Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: cc @machadoum |
) (#111845) Co-authored-by: Pablo Machado <[email protected]>
) (#111844) Co-authored-by: Pablo Machado <[email protected]>
issue: #111679
Summary
During a previous fix, I broke the trend graph for External alerts. The reason is that I started counting the total of events based on the bucket. But every kind of
MatrixHistogramType
has a different aggregation name.eventActionGroup
was wrongly hardcoded insideuseMatrixHistogram
. After this change,buckets
should be returned for every type of MatrixHistogram.132640542-440cd073-2c02-4277-8226-d2fd7c2ead8b.mp4
Checklist