[Security Solution][Analyzer] Fix graph overlay persist despite filter group changes

[christineweng]

This PR contain fixes for issue:

• [Security Solution] Resolver is displayed when navigate from open state tab to closed or acknowledged state #122178

Summary

The analyzer (graph overlay) was shown despite the filter (open, acknowledged, or closed) changes. filterStatus is now passed to the useTimelineEvents to monitor change in filter.

Note that code is modified at the t-grid level (deeper) instead of the alerts table (i.e. onFilterChange). For instance, when switching from open to acknowledged, data fetching takes some time and the previous events table is still showing with loading animation. Disabling graph overlay before the new data is fully loaded causes a quick flash, hence the check is done in t-grid and disabling graph overlay only after the new data is fetched. This enables a better user experience.

This bug affects 2 pages, and both are now fixed:

Alerts in Alerts page

Screen.Recording.2022-10-31.at.5.43.27.PM.mov

Alerts in Rule Detail page

Screen.Recording.2022-10-31.at.5.44.34.PM.mov

[elasticmachine]

Pinging @elastic/security-threat-hunting (Feature:Resolver)

[elasticmachine]

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

[michaelolo24]

Thanks for fixing this!

[kibana-ci]

💛 Build succeeded, but was flaky

• Buildkite Build
• Commit: 0df6296

Failed CI Steps

• Security Solution Tests #1
• Security Solution Tests #4

Test Failures

• [job] [logs] Security Solution Tests #1 / Alerts timeline Privileges: read only "before each" hook for "should not allow user with read only privileges to attach alerts to existing cases"
• [job] [logs] Security Solution Tests #1 / Cases Creates a new case with timeline and opens the timeline
• [job] [logs] Security Solution Tests #4 / url state sets and reads the url state for timeline by id

Metrics [docs]

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
timelines	348	349	+1

Async chunks

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

id	before	after	diff
timelines	74.0KB	74.2KB	+224.0B

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id	before	after	diff
timelines	32	33	+1

Unknown metric groups

API count

id	before	after	diff
timelines	457	458	+1

ESLint disabled in files

id	before	after	diff
osquery	1	2	+1

ESLint disabled line counts

id	before	after	diff
enterpriseSearch	19	21	+2
fleet	58	64	+6
osquery	108	113	+5
securitySolution	440	446	+6
total			+19

Total ESLint disabled count

id	before	after	diff
enterpriseSearch	20	22	+2
fleet	66	72	+6
osquery	109	115	+6
securitySolution	517	523	+6
total			+20

History

• 💚 Build #84915 succeeded 442e3e5
• 💛 Build #84233 was flaky d6a1fb8
• 💔 Build #84214 failed ec55ac1

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

cc @christineweng