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][9.0] Creates Automatic Migrations guide #713

Merged
merged 22 commits into from
Apr 4, 2025

Conversation

benironside
Copy link
Contributor

@benironside benironside commented Mar 7, 2025

Addresses the 9.0 component of #586 by creating a new guide for the AI-powered SIEM migrations feature.

Previews: Automatic Migration (new page)

@benironside benironside added documentation Improvements or additions to documentation enhancement New feature or request labels Mar 7, 2025
@benironside benironside self-assigned this Mar 7, 2025
@Charelzard
Copy link

Ben this looks great! Let me know once the images are updated and I'll rereview. Im thinking we do a video or gif for a full walkthrough, thoughts?

My feedback is as follows.

  • Instead of referencing the tool tip for the types of statuses we should list them all for installed, translated partial, etc.
  • I updated the RBAC documenation section to include a sentence and screen shot.
  • @bojanasan Sent over an update in Figma for Partially translated which could be explained further in the docs for how a user can get a rule to a green state. There are three main areas:
  1. User needs to define an index pattern when editing.
  2. A macro or lookup is not defined
  3. Its an actual syntax error.

I think we could either put this in the doc or within the text of the page. Let me know what would be better here.

Copy link

@Charelzard Charelzard left a comment

Choose a reason for hiding this comment

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

See all requested updates. I do think we might need a video or gif to show the whole process beginning to end but If you think the process clearly describes how to navigate the entire process we can defer to the current docs. For some reason the existing pages dont show the images yet.

@Charelzard
Copy link

I dont see the Overview tab or summary tabs along with the index information. FYI @benironside

@Charelzard
Copy link

Charelzard commented Mar 26, 2025

@benironside Please update Step 2 to this:

Elastic ships with a default LLM, if you prefer to change the default model under Configure AI provider select a model, or add a new one. For information on how different models perform, refer to the LLM performance matrix.

@benironside benironside changed the title [security][9.0] Creates SIEM migrations guide [security][9.0] Creates Automatic Migrations guide Apr 3, 2025
Copy link
Contributor

@nastasha-solomon nastasha-solomon left a comment

Choose a reason for hiding this comment

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

This is really cool - great job, @benironside! I left some suggestions and minor edits throughout. Feel free to ping me once you've gone through and made the changes that you want.

Co-authored-by: Nastasha Solomon <79124755+nastasha-solomon@users.noreply.github.com>
benironside and others added 3 commits April 3, 2025 15:16
Co-authored-by: Nastasha Solomon <79124755+nastasha-solomon@users.noreply.github.com>
@benironside benironside enabled auto-merge (squash) April 4, 2025 21:14
@benironside benironside merged commit d712385 into main Apr 4, 2025
4 of 5 checks passed
@benironside benironside deleted the 586-siem-migration-guide branch April 4, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants