[SecuritySolution] Setup dashboard view page

[angorayc]

Summary

#152955

Demo link: https://kibana-pr-154806.kb.us-west2.gcp.elastic-cloud.com:9243/
https://p.elstc.co/paste/vPoSt7eC#txCuL6KJWRCPRJ0ae4minlT7IgGBJ5F8MTloWIOVQcw

This create a single dashboard view under Security Solution:

• Add dashboard view path: /app/security/dashboards/:dashboardId
• Move the dashboards landing page to this new sub-plugin.
• Check users' read permission to render a dashboard.
• Render a dashboard with the given saved object id.
• Show the dashboard name in the breadcrumbs.

Dashboard not found:

Dashboard rendered:

Interact with filters and query:

Screen.Recording.2023-03-23.at.13.47.06.mov

Interact with Open in Lens and Investigate in timeline

Screen.Recording.2023-03-28.at.12.07.32.mov

Steps to verify:

1. Create a dashboard from /app/dashboards#/list, save it and copy the dashboard saved object id from url.
2. Visit /app/security/dashboards/:dashboardId

Known issues:
#154842
#154843

Checklist

Delete any items that are not applicable to this PR.

• Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
• Unit or functional tests were updated or added to match the most common scenarios

[angorayc]

Adding this because the time range of the Visualization wouldn't update after brushing on a time-based histogram / area chart.

brushing_on_chart.mov

After:

Screen.Recording.2023-03-27.at.11.35.47.mov

[elasticmachine]

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

[elasticmachine]

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

[stratoula]

@angorayc I don't understand why onBrush doesn't work tbh. It works on all visualizations on a dashboard without adding an extra method on unified search. Are you using the onBrushEnd method of Lens embeddable? I think it makes sense first to understand why it doesn't work cc @dej611 if you have any input here.

[dej611]

@angorayc I don't understand why onBrush doesn't work tbh. It works on all visualizations on a dashboard without adding an extra method on unified search. Are you using the onBrushEnd method of Lens embeddable? I think it makes sense first to understand why it doesn't work cc @dej611 if you have any input here.

Agree with @stratoula that it is better to find out what's going wrong here rather than adding a new method.
I cannot find anything strange from the code yet, but I see it is using an Embedded Dashboard in this page solution rather than a Lens Embeddable. I'll try to investigate on that route.

[stratoula]

cc @ThomThomson for his input here too.

[angorayc]

I think the reason that brushing doesn't work because Security Solution doesn't share the same url param structure as others, we do not put time range under _g, and we do not use kbnUrlStateStorage. We sync the url state when query changed by dispatching an action to reducer. If the action isn't triggered when query changed, the time range passed to dashboard wouldn't update.

I was thinking of calling onQueryChange on Line 191, but eventually adding a new prop onTimeRangeChange to be more specific about the date range changed here. What do you think if we remove onTimeRangeChange and just call onQueryChange on src/plugins/unified_search/public/search_bar/search_bar.tsx line 191?

[stratoula]

@angorayc yes I think using the onQueryChange makes more sense to me, do you want to give it a try?

[angorayc]

@stratoula , yup, changed onTimeRangeChange to onQueryChange, but still need to call it in src/plugins/unified_search/public/search_bar/search_bar.tsx line 191

[stratoula]

I see, no I suggest to try understand why onBrush doesnt work for embedded dashboards. Let's see if Marco and Devon find anything otherwise I have to check the code.

[angorayc]

@ferenrigue , @paulewing

Here's a link to the demo page: https://kibana-pr-153040.kb.us-west2.gcp.elastic-cloud.com:9243/app/security/dashboards/6294c960-ce35-11ed-b8ca-51636b04063c?sourcerer=(default:(id:security-solution-default,selectedPatterns:!(%27auditbeat-*%27,%27logs-*%27)))

https://p.elstc.co/paste/pOFVo-fV#Zgp3hnsnijsDHbki4y9Cy5F+apet-hYEcedpDzsc+f7

The edit button takes user to the dashboards app at the moment. Eventually we should be able to allow users to edit in SecuritySolution’s app.
And the entry of the dashboard view page should be here, will add it later when we think it's ready:

[stratoula]

Thanx Angela, unified search changes LGTM!

[semd]

Awesome! Thanks for addressing the comments @angorayc
This Dashboard view is super powerful 🔥🔥
LGTM! 🚀

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 54472ed
• Cloud Deployment

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
securitySolution	3792	3806	+14

Async chunks

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

id	before	after	diff
securitySolution	15.9MB	15.9MB	+6.8KB
unifiedSearch	267.5KB	267.6KB	+155.0B
total			+6.9KB

Page load bundle

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

id	before	after	diff
securitySolution	58.0KB	58.1KB	+46.0B

Unknown metric groups

ESLint disabled in files

id	before	after	diff
securitySolution	80	79	-1

ESLint disabled line counts

id	before	after	diff
securitySolution	433	436	+3

Total ESLint disabled count

id	before	after	diff
securitySolution	513	515	+2

History

• 💛 Build #118975 was flaky 7e48271
• 💚 Build #117783 succeeded 7c70a98
• 💔 Build #117721 failed fd2366b820b3e54c4639d71acb318a58d95cba1a
• 💚 Build #117427 succeeded 8c5a00446c4ab322adb0685181b641713e0cfbaa
• 💛 Build #116791 was flaky fd257cc43a31bf9056981c8d96272b0c52bd874a

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

cc @angorayc