Skip to content

[SecuritySolutions] Fix Entity Analytics dashboard fires many risk_score/index_status requests#179510

Merged
machadoum merged 3 commits intoelastic:mainfrom
machadoum:siem-ea-178838
Apr 2, 2024
Merged

[SecuritySolutions] Fix Entity Analytics dashboard fires many risk_score/index_status requests#179510
machadoum merged 3 commits intoelastic:mainfrom
machadoum:siem-ea-178838

Conversation

@machadoum
Copy link
Copy Markdown
Member

@machadoum machadoum commented Mar 27, 2024 •
edited
Loading

Summary

The Entity Analytics Dashboard page called the /risk_score/index_status API many times, which led to a very long loading time.

I wrapped all API calls inside useEntityAnalyticsRoute with useMemo to quick-fix the multiple unnecessary HTTP calls.

Checklist

Delete any items that are not applicable to this PR.

@machadoum machadoum self-assigned this Mar 27, 2024
@machadoum machadoum added technical debt Improvement of the software architecture and operational architecture Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Entity Analytics Security Entity Analytics Team bug Fixes for quality problems that affect the customer experience and removed bug Fixes for quality problems that affect the customer experience labels Mar 27, 2024
@machadoum machadoum marked this pull request as ready for review March 27, 2024 16:52
@machadoum machadoum requested a review from a team as a code owner March 27, 2024 16:52
@machadoum machadoum requested a review from tiansivive March 27, 2024 16:52
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 27, 2024

Pinging @elastic/security-solution (Team: SecuritySolution)

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 27, 2024

Pinging @elastic/security-entity-analytics (Team:Entity Analytics)

hop-dev
hop-dev approved these changes Mar 28, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@hop-dev hop-dev left a comment

Choose a reason for hiding this comment

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

🚀

@machadoum
Copy link
Copy Markdown
Member Author

machadoum commented Mar 28, 2024

/ci

@machadoum machadoum enabled auto-merge (squash) April 2, 2024 07:39
@machadoum machadoum merged commit e290d2c into elastic:main Apr 2, 2024
@kibana-ci
Copy link
Copy Markdown

kibana-ci commented Apr 2, 2024

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #34 / Rules Management - Prebuilt Rules - Bundled Prebuilt Rules Package @ess @serverless @skipInQA install_bundled_prebuilt_rules should install prebuilt rules from the package that comes bundled with Kibana

Metrics [docs]

Async chunks

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

id before after diff
securitySolution 16.6MB 16.6MB +39.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @machadoum

@kibanamachine kibanamachine added v8.14.0 backport:skip This PR does not require backporting labels Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hop-dev hop-dev hop-dev approved these changes

@tiansivive tiansivive Awaiting requested review from tiansivive tiansivive was automatically assigned from elastic/security-entity-analytics

Assignees

@machadoum machadoum

Labels

backport:skip This PR does not require backporting release_note:fix Team:Entity Analytics Security Entity Analytics Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. technical debt Improvement of the software architecture and operational architecture v8.14.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@machadoum @elasticmachine @kibana-ci @hop-dev @kibanamachine