Skip to content

[Security] Adds section about Elastic Managed LLM to Security AI assistant #1778

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Jun 19, 2025
Merged

[Security] Adds section about Elastic Managed LLM to Security AI assistant #1778

merged 7 commits into from
Jun 19, 2025

Conversation

szabosteve
Copy link
Contributor

@szabosteve szabosteve commented Jun 18, 2025 •
edited
Loading

Overview

Related to #1754

This PR adds a section about the Elastic Managed LLM to the Security AI Assistant docs.

PREVIEW

@szabosteve szabosteve requested review from dhru42 and shubhaat June 18, 2025 08:37
@szabosteve szabosteve added the documentation Improvements or additions to documentation label Jun 18, 2025
@szabosteve szabosteve requested a review from a team as a code owner June 18, 2025 08:37
@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 18, 2025 •
edited
Loading

🔍 Preview links for changed docs:

🔔 The preview site may take up to 3 minutes to finish building. These links will become live once it completes.

bmorelli25
bmorelli25 reviewed Jun 19, 2025
View reviewed changes
shubhaat
shubhaat approved these changes Jun 19, 2025
View reviewed changes
Copy link
Contributor

@shubhaat shubhaat left a comment

Choose a reason for hiding this comment

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

lgtm.

szabosteve
szabosteve commented Jun 19, 2025
View reviewed changes
natasha-moore-elastic
natasha-moore-elastic approved these changes Jun 19, 2025
View reviewed changes
Copy link
Contributor

@natasha-moore-elastic natasha-moore-elastic left a comment

Choose a reason for hiding this comment

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

Looks good, just one nit, thanks!

@szabosteve szabosteve merged commit a8f7e8b into main Jun 19, 2025
7 checks passed
@szabosteve szabosteve deleted the szabosteve/emllm-security-ai-assistant branch June 19, 2025 10:06
@natasha-moore-elastic natasha-moore-elastic mentioned this pull request Jun 27, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bmorelli25 bmorelli25 bmorelli25 left review comments

@natasha-moore-elastic natasha-moore-elastic natasha-moore-elastic approved these changes

@dhru42 dhru42 Awaiting requested review from dhru42

+1 more reviewer

@shubhaat shubhaat shubhaat approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation ml nlp v9.1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@szabosteve @shubhaat @bmorelli25 @natasha-moore-elastic