[SecuritySolution] Replace donuts with Lens

[angorayc]

Summary

This pr is a part of #147261: Replace donut charts with Lens:
This is behind feature flag chartEmbeddablesEnabled

Items to verify:

1. If the queries and filters from the global search box are applied.
2. If the correct alert index is applied by different spaced.
3. Visualization actions working correctly.

Detection and Response dashboard:

Host details:

User details:

Network details:

Known issues:

1. Not showing legend for alerts donut charts at the moment: There is a logic in Lens that it doesn't show the legend item if its value is zero. - It doesn't show the legend item if its value is zero #149681
2. JS warnings triggered by incorrect state changed. Fixed by [Security Solution] Resolve JS warnings triggered by incorrect state changed #148552
3. No label in the donut by default when open in Lens - Lens doesn't support displaying a label in the middle of the donut chart by default, so it is currently available in Security Solution. - [Lens] An option to display title / subtitle for charts #149675
4. Applying filters or extra action while clicking the donut is not available at the moment. - [Lens][Embeddable] Provide a way to override default action on trigger #149027
5. Page crashes after editing the filter applied from a Lens Embeddable - [SecuritySolution] Page crashes after editing the filter applied from a Lens Embeddable. #148710

Checklist

Delete any items that are not applicable to this PR.

• Unit or functional tests were updated or added to match the most common scenarios

[elasticmachine]

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

[elasticmachine]

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

[jamster10]

if isChartEmbeddablesEnabled is true though, this component wouldnt render as As AlertDonutEmbeddable would :P. Same for the instances below it. I think you can remove the prop.

[angorayc]

OK, I have this removed in this PR: https://github.com/elastic/kibana/pull/147261/files

[jamster10]

Hey @angorayc this looks great. I went through the code as well as tested the build and it all looks good.

Some questions:
It looks like youre trying to swap out all usage of DonutCharts in favour of lens embeddables, leaving the only usage of DonutChart on the Alerts page and Risk score. Is that because of the lack of legend?

Also, am i right in assuming the default behaviour for lens embeddable clicks is to add a filter. Is that something we can plug in to? Curious because it has ramifications for something I was working on- routing to the Alert page on click from the donut charts on the Explore pages.

Finally, it looks like the bug Steph fixed might impact [useRefetchByRestartingSession](https://github.com/elastic/kibana/blob/ee27615e644cf391b7b04ea44f1d3fc88bf6031c/x-pack/plugins/security_solution/public/explore/containers/use_refetch_by_session.tsx) I dont think in an adverse way though.

[angorayc]

Hey @angorayc this looks great. I went through the code as well as tested the build and it all looks good.

Some questions: It looks like youre trying to swap out all usage of DonutCharts in favour of lens embeddables, leaving the only usage of DonutChart on the Alerts page and Risk score. Is that because of the lack of legend?

Because this PR has lots of files changed already, so I'd like to replace them in a follow up PR.

Also, am i right in assuming the default behaviour for lens embeddable clicks is to add a filter. Is that something we can plug in to? Curious because it has ramifications for something I was working on- routing to the Alert page on click from the donut charts on the Explore pages.

Yup, should be able to do that. I'll put it on the list.

Finally, it looks like the bug Steph fixed might impact [useRefetchByRestartingSession](https://github.com/elastic/kibana/blob/ee27615e644cf391b7b04ea44f1d3fc88bf6031c/x-pack/plugins/security_solution/public/explore/containers/use_refetch_by_session.tsx) I dont think in an adverse way though.

Thanks for the heads up, I'm looking to that with Steph.

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 7d03079

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
securitySolution	3462	3466	+4

Async chunks

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

id	before	after	diff
securitySolution	12.7MB	12.7MB	+39.1KB

History

• 💔 Build #98947 failed 7867014
• 💛 Build #98887 was flaky 7479091
• 💔 Build #98822 failed 1ce2b7d
• 💔 Build #98767 failed 3b999cc
• 💔 Build #98713 failed 8a84964

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

cc @angorayc

[dasansol92]

Only checked linter changes on x-pack/plugins/security_solution/public/management/cypress/tsconfig.json

[dasansol92]

Approving linter changes from management folder