Skip to content

[AI4SOC] AI settings page#217373

Merged
stephmilovic merged 26 commits intoelastic:mainfrom
stephmilovic:ai_settings_page
Apr 21, 2025
Merged

[AI4SOC] AI settings page#217373
stephmilovic merged 26 commits intoelastic:mainfrom
stephmilovic:ai_settings_page

Conversation

@stephmilovic
Copy link
Contributor

@stephmilovic stephmilovic commented Apr 7, 2025

Summary

Completes the AI Settings page for AI for SOC.

Screenshot 2025-04-07 at 10 51 48 AM - Hides Stack Management AI Assistant settings when `product_line: 'ai_soc', product_tier: 'search_ai_lake'` - Links to new `SearchAILakeConfigurationsSettingsManagement` page from Assistant settings context menu

To test

  • Add the following settings to serverless.security.dev.yml:

    xpack.securitySolutionServerless.productTypes:
    [
      { product_line: 'ai_soc', product_tier: 'search_ai_lake' },
    ]
    
  • Run Kibana serverless

  • Navigate through each AI settings tab and ensure results are matching the mock

  • Ensure Stack Management => Security AI Settings does not show

  • Ensure the Assistant settings context menu links to the new Configurations AI Settings when clicking AI Assistant settings and Knowledge Base
    Screenshot 2025-04-08 at 9 10 55 AM

@stephmilovic stephmilovic added release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Security Generative AI Security Generative AI backport:version Backport to applied version labels v9.1.0 v8.19.0 and removed v8.19.0 labels Apr 7, 2025
@stephmilovic stephmilovic marked this pull request as ready for review April 8, 2025 18:50
@stephmilovic stephmilovic requested review from a team as code owners April 8, 2025 18:50
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@tomsonpl tomsonpl left a comment

Choose a reason for hiding this comment

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

Left some nit comments, tested locally and everything seems to work as expected, great job 👍

@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

1 similar comment
@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

Copy link
Contributor

@semd semd left a comment

Choose a reason for hiding this comment

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

LGTM!

@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

@angorayc
Copy link
Contributor

One more thing Steph, could you please double check if we still have the add integration link at the top right corner? I think we can remove it if it's there: https://www.figma.com/design/DYs7j4GQdAhg7aWTLI4R69/AI4DSOC?node-id=3747-54754&m=dev

Screenshot 2025-04-11 at 15 27 55

@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

@stephmilovic stephmilovic enabled auto-merge (squash) April 21, 2025 14:56
@stephmilovic stephmilovic merged commit 361d38a into elastic:main Apr 21, 2025
9 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
automaticImport 752 799 +47
securitySolution 7265 7269 +4
total +51

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
@kbn/elastic-assistant 146 149 +3

Async chunks

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

id before after diff
securitySolution 9.0MB 9.0MB +2.6KB

Page load bundle

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

id before after diff
securitySolution 89.4KB 89.6KB +139.0B
Unknown metric groups

API count

id before after diff
@kbn/elastic-assistant 179 185 +6

History

pgayvallet pushed a commit to pgayvallet/kibana that referenced this pull request Apr 22, 2025
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 23, 2025
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 217373 locally

@stephmilovic stephmilovic added the backport:skip This PR does not require backporting label Apr 23, 2025
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 23, 2025
@stephmilovic stephmilovic removed the backport:version Backport to applied version labels label Apr 23, 2025
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request May 29, 2025
stephmilovic added a commit to PhilippeOberti/kibana that referenced this pull request Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Security Generative AI Security Generative AI Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants