Skip to content

[8.x] [Observability] Related alerts based on scoring !! (#215673) #218664

Merged
shahzad31 merged 7 commits into
elastic:8.19from
dominiqueclarke:backport/8.x/pr-215673
Apr 22, 2025
Merged

[8.x] [Observability] Related alerts based on scoring !! (#215673) #218664
shahzad31 merged 7 commits into
elastic:8.19from
dominiqueclarke:backport/8.x/pr-215673

Conversation

@dominiqueclarke
Copy link
Copy Markdown
Contributor

@dominiqueclarke dominiqueclarke commented Apr 18, 2025

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

shahzad31 and others added 5 commits April 15, 2025 09:57
@shahzad31
[Observability] Related alerts based on scoring !! (elastic#215673)
be19265 
## Summary

Copying most of elastic#214017 !!

Fixes elastic#214372

### Implementation
We are now using response ops alerts table with custom score querying
based on tags/groups matches and Jaccard similarity on documents !!

<img width="1728" alt="image"
src="https://github.com/user-attachments/assets/b3a69280-c05d-4100-be6a-2c8dadcc051d"
/>

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Dominique Clarke <dominique.clarke@elastic.co>
Co-authored-by: Kevin Delemme <kevin.delemme@elastic.co>
(cherry picked from commit 760106e)

# Conflicts:
#	x-pack/solutions/observability/plugins/observability/public/components/alert_actions/alert_actions.tsx
#	x-pack/solutions/observability/plugins/observability/public/pages/alert_details/components/related_alerts.test.tsx
#	x-pack/solutions/observability/plugins/observability/public/pages/alert_details/components/related_alerts.tsx
#	x-pack/solutions/security/plugins/security_solution/public/detections/components/alert_summary/table/render_cell.test.tsx
#	x-pack/solutions/security/plugins/security_solution/public/detections/components/alert_summary/table/render_cell.tsx
@dominiqueclarke
add view details i18n
1a81170
@dominiqueclarke
adjust tests
63ddb74
@shahzad31
Merge branch 'old-8.x' into backport/8.x/pr-215673
b2da9fe
@dominiqueclarke
adjust jest tests
dc82c89
@botelastic botelastic Bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label Apr 18, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 18, 2025

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@dominiqueclarke dominiqueclarke added the backport This PR is a backport of another PR label Apr 18, 2025
@dominiqueclarke dominiqueclarke mentioned this pull request Apr 18, 2025
Closed
justinkambic
justinkambic approved these changes Apr 18, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@justinkambic justinkambic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM WFG

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 21, 2025

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
observability 1242 1245 +3

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/alerting-types 249 250 +1
observability 753 754 +1
total +2

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cases 1.3MB 1.3MB +208.0B
embeddableAlertsTable 796.3KB 796.5KB +204.0B
ml 5.4MB 5.4MB +204.0B
observability 1.3MB 1.3MB -889.0B
securitySolution 9.2MB 9.2MB +205.0B
triggersActionsUi 1.4MB 1.4MB +205.0B
total +137.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
observability 94.0KB 93.8KB -209.0B
Unknown metric groups

API count

id before after diff
@kbn/alerting-types 253 254 +1

async chunk count

id before after diff
observability 27 24 -3

ESLint disabled line counts

id before after diff
observability 48 45 -3

Total ESLint disabled count

id before after diff
observability 49 46 -3

History

@shahzad31 shahzad31 merged commit de44716 into elastic:8.19 Apr 22, 2025
8 checks passed
@shahzad31 shahzad31 deleted the backport/8.x/pr-215673 branch April 22, 2025 05:29
@mykolaharmash mykolaharmash mentioned this pull request Apr 22, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@justinkambic justinkambic justinkambic approved these changes

@kibanamachine kibanamachine Awaiting requested review from kibanamachine kibanamachine is a code owner

Assignees

No one assigned

Labels

backport This PR is a backport of another PR Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dominiqueclarke @elasticmachine @justinkambic @shahzad31