Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Security Solution][PoC] Discover table in Explore pages #166104

Closed

Conversation

semd
Copy link
Contributor

@semd semd commented Sep 8, 2023

Summary

PoC that implements the integration the @kbn/unified-data-table (Discover grid) in all the event tables of Explore pages (Hosts, Network, Users)

Captura de pantalla 2023-09-08 a les 18 37 02
  • Data fetching
  • Data convertion
  • Extend bulk actions with security bulk actions
  • Configure row renderers correctly
  • Add Field Browser
  • Column order changes

@semd semd requested review from a team as code owners September 8, 2023 16:34
@semd semd marked this pull request as draft September 8, 2023 16:34
@semd semd self-assigned this Sep 8, 2023
@semd semd changed the title [PoC] Discover table in Explore pages [Security Solution][PoC] Discover table in Explore pages Sep 8, 2023
@kibana-ci
Copy link
Collaborator

kibana-ci commented Sep 8, 2023

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Serverless Security Investigations Cypress Tests #1 / Bulk Investigate in Timeline Host -> Events Viewer "before each" hook for "Adding multiple events to the timeline should be successful" "before each" hook for "Adding multiple events to the timeline should be successful"
  • [job] [logs] Investigations - Security Solution Cypress Tests #6 / Bulk Investigate in Timeline Host -> Events Viewer "before each" hook for "Adding multiple events to the timeline should be successful" "before each" hook for "Adding multiple events to the timeline should be successful"
  • [job] [logs] Serverless Security Investigations Cypress Tests #1 / Bulk Investigate in Timeline Host -> Events Viewer "before each" hook for "Adding multiple events to the timeline should be successful" "before each" hook for "Adding multiple events to the timeline should be successful"
  • [job] [logs] Investigations - Security Solution Cypress Tests #6 / Bulk Investigate in Timeline Host -> Events Viewer "before each" hook for "Adding multiple events to the timeline should be successful" "before each" hook for "Adding multiple events to the timeline should be successful"
  • [job] [logs] Serverless Security Investigations Cypress Tests #1 / Bulk Investigate in Timeline Host -> Sessions Viewer "before each" hook for "Adding multiple events to the timeline should be successful" "before each" hook for "Adding multiple events to the timeline should be successful"
  • [job] [logs] Investigations - Security Solution Cypress Tests #6 / Bulk Investigate in Timeline Host -> Sessions Viewer "before each" hook for "Adding multiple events to the timeline should be successful" "before each" hook for "Adding multiple events to the timeline should be successful"
  • [job] [logs] Serverless Security Investigations Cypress Tests #1 / Bulk Investigate in Timeline Host -> Sessions Viewer "before each" hook for "Adding multiple events to the timeline should be successful" "before each" hook for "Adding multiple events to the timeline should be successful"
  • [job] [logs] Investigations - Security Solution Cypress Tests #6 / Bulk Investigate in Timeline Host -> Sessions Viewer "before each" hook for "Adding multiple events to the timeline should be successful" "before each" hook for "Adding multiple events to the timeline should be successful"
  • [job] [logs] Security Solution Cypress Tests #12 / EQL rules Detection rules, EQL Creates and enables a new EQL rule Creates and enables a new EQL rule
  • [job] [logs] Security Solution Cypress Tests #12 / EQL rules Detection rules, EQL Creates and enables a new EQL rule Creates and enables a new EQL rule
  • [job] [logs] Security Solution Cypress Tests #12 / EQL rules Detection rules, sequence EQL Creates and enables a new EQL rule with a sequence Creates and enables a new EQL rule with a sequence
  • [job] [logs] Security Solution Cypress Tests #12 / EQL rules Detection rules, sequence EQL Creates and enables a new EQL rule with a sequence Creates and enables a new EQL rule with a sequence
  • [job] [logs] Explore - Security Solution Cypress Tests #1 / Events Viewer Events behavior "before each" hook for "filters the events by applying filter criteria from the search bar at the top of the page" "before each" hook for "filters the events by applying filter criteria from the search bar at the top of the page"
  • [job] [logs] Explore - Security Solution Cypress Tests #1 / Events Viewer Events behavior "before each" hook for "filters the events by applying filter criteria from the search bar at the top of the page" "before each" hook for "filters the events by applying filter criteria from the search bar at the top of the page"
  • [job] [logs] Explore - Security Solution Cypress Tests #1 / Events Viewer Events viewer fields behaviour "before each" hook for "adds a field to the events viewer when the user clicks the checkbox" "before each" hook for "adds a field to the events viewer when the user clicks the checkbox"
  • [job] [logs] Explore - Security Solution Cypress Tests #1 / Events Viewer Events viewer fields behaviour "before each" hook for "adds a field to the events viewer when the user clicks the checkbox" "before each" hook for "adds a field to the events viewer when the user clicks the checkbox"
  • [job] [logs] Explore - Security Solution Cypress Tests #1 / Events Viewer Fields rendering "before each" hook for "displays "view all" option by default" "before each" hook for "displays "view all" option by default"
  • [job] [logs] Explore - Security Solution Cypress Tests #1 / Events Viewer Fields rendering "before each" hook for "displays "view all" option by default" "before each" hook for "displays "view all" option by default"
  • [job] [logs] Defend Workflows Cypress Tests #2 / Form User with access can edit and delete an endpoint response action delete response action inside of a rule delete response action inside of a rule
  • [job] [logs] Defend Workflows Cypress Tests #2 / Form User without access can not edit, add nor delete an endpoint response action All response action controls are disabled All response action controls are disabled
  • [job] [logs] Explore - Security Solution Cypress Tests #2 / Pagination Host uncommon processes table) pagination keeps track of page results when tabs change pagination keeps track of page results when tabs change
  • [job] [logs] Explore - Security Solution Cypress Tests #2 / Pagination Host uncommon processes table) pagination keeps track of page results when tabs change pagination keeps track of page results when tabs change
  • [job] [logs] Investigations - Security Solution Cypress Tests #2 / persistent timeline "before all" hook for "persist the deletion of a column" "before all" hook for "persist the deletion of a column"
  • [job] [logs] Investigations - Security Solution Cypress Tests #2 / persistent timeline "before all" hook for "persist the deletion of a column" "before all" hook for "persist the deletion of a column"
  • [job] [logs] Jest Tests #12 / PreviewHistogram when there is no data it renders an empty histogram and table

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 4510 4594 +84

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.6MB 12.7MB +59.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 62.1KB 62.2KB +116.0B
Unknown metric groups

async chunk count

id before after diff
securitySolution 50 51 +1

History

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

cc @semd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants