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

Adds RAG for alerts info to the AI Assistant page #4520

Merged
merged 15 commits into from
Jan 3, 2024

Conversation

benironside
Copy link
Contributor

@benironside benironside commented Dec 21, 2023

Fixes #4456. Updates the AI Assistant page with information about the RAG for alerts feature that allows you to send multiple alerts as part of each prompt, so that AI Assistant can answer questions about recent alerts in your environment rather than just individual alerts.

Preview: AI Assistant

Related updates to the "Triage alerts with AI Assistant" page can be previewed here: #4359 . (RAG for Alerts changes are limited to the new first section "Use AI Assistant to Triage multiple alerts".)

@benironside benironside added enhancement New feature or request Feature: Elastic AI Assistant Interface for interacting with generative AIs v8.12.0 labels Dec 21, 2023
@benironside benironside self-assigned this Dec 21, 2023
@benironside benironside requested a review from a team as a code owner December 21, 2023 20:34
Copy link

A documentation preview will be available soon.

Help us out by validating the Buildkite preview and reporting issues here.
Please also be sure to double check all images to ensure they are correct in the preview.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

Choose a reason for hiding this comment

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

Thanks @benironside for your help with the Alerts setting!
Your feedback is included in elastic/kibana#173809

Copy link

@andrew-goldstein andrew-goldstein left a comment

Choose a reason for hiding this comment

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

Thanks @benironside for these updates to the KB section
✅ Desk tested via the preview link
LGTM 🚀

Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

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

A few edits here, and two other things:

  • The lead-in on line 147 says there are "four tabs" but now there are five. We should just omit numbering like this, which can easily get outdated and overlooked like this was, and just say something like "the following tabs."

  • The section on the tabs has gotten quite long and jumbly, and I think we need a few subsections for the more in-depth config settings. We could have a simple overview bullet list of the five tabs, and then link to the subsections for more details.

docs/assistant/security-assistant.asciidoc Outdated Show resolved Hide resolved
docs/assistant/security-assistant.asciidoc Outdated Show resolved Hide resolved
docs/assistant/security-assistant.asciidoc Outdated Show resolved Hide resolved
docs/assistant/security-assistant.asciidoc Outdated Show resolved Hide resolved
@benironside benironside requested a review from a team January 3, 2024 19:09
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.

Left a handful of light edits. All around, lgtm!

docs/assistant/security-assistant.asciidoc Outdated Show resolved Hide resolved
docs/assistant/security-assistant.asciidoc Outdated Show resolved Hide resolved
docs/assistant/security-assistant.asciidoc Outdated Show resolved Hide resolved
@benironside benironside merged commit 938e687 into main Jan 3, 2024
4 checks passed
mergify bot pushed a commit that referenced this pull request Jan 3, 2024
* Adds info about RAG for alerts

* adds section tag

* Adds an image

* tweaks formatting

* first pass at incorporating Joe's feedback

* second pass at Joe's feedback

* minor formatting fix

* changes headers

* Update docs/assistant/security-assistant.asciidoc

Co-authored-by: Nastasha Solomon <[email protected]>

* Update docs/assistant/security-assistant.asciidoc

Co-authored-by: Nastasha Solomon <[email protected]>

* Update docs/assistant/security-assistant.asciidoc

Co-authored-by: Nastasha Solomon <[email protected]>

---------

Co-authored-by: Nastasha Solomon <[email protected]>
(cherry picked from commit 938e687)
benironside added a commit that referenced this pull request Jan 3, 2024
* Adds info about RAG for alerts

* adds section tag

* Adds an image

* tweaks formatting

* first pass at incorporating Joe's feedback

* second pass at Joe's feedback

* minor formatting fix

* changes headers

* Update docs/assistant/security-assistant.asciidoc

Co-authored-by: Nastasha Solomon <[email protected]>

* Update docs/assistant/security-assistant.asciidoc

Co-authored-by: Nastasha Solomon <[email protected]>

* Update docs/assistant/security-assistant.asciidoc

Co-authored-by: Nastasha Solomon <[email protected]>

---------

Co-authored-by: Nastasha Solomon <[email protected]>
(cherry picked from commit 938e687)

Co-authored-by: Benjamin Ironside Goldstein <[email protected]>
@benironside benironside mentioned this pull request Jan 15, 2024
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Feature: Elastic AI Assistant Interface for interacting with generative AIs v8.12.0
Projects
Development

Successfully merging this pull request may close these issues.

[Request] Retrieval Augemented Generation (RAG) for Alerts
4 participants