Skip to content

Security Discover profile#5638

Merged
natasha-moore-elastic merged 6 commits intomainfrom
issue-3021-sec-profile
Mar 30, 2026
Merged

Security Discover profile#5638
natasha-moore-elastic merged 6 commits intomainfrom
issue-3021-sec-profile

Conversation

@natasha-moore-elastic
Copy link
Copy Markdown
Contributor

@natasha-moore-elastic natasha-moore-elastic commented Mar 26, 2026

Summary

Resolves #3021 by documenting the Security Discover profile.

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Cursor

@natasha-moore-elastic natasha-moore-elastic self-assigned this Mar 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 26, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 26, 2026

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@natasha-moore-elastic natasha-moore-elastic marked this pull request as ready for review March 26, 2026 11:28
@natasha-moore-elastic natasha-moore-elastic requested review from a team as code owners March 26, 2026 11:28
Copy link
Copy Markdown
Contributor

@benironside benironside left a comment

Choose a reason for hiding this comment

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

Nice work. What do you think about adding a reference to this page to /solutions/security/investigate.md (section landing page)? I think it would help folks to see how Discover can fit into the wider context of our tools.

@logeekal
Copy link
Copy Markdown

Thanks @natasha-moore-elastic , docs look good to me but don't we need any screenshots for the security profile?

Copy link
Copy Markdown
Contributor

@davismcphee davismcphee left a comment

Choose a reason for hiding this comment

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

Changes look good overall from the Discover side, just left one suggestion I think is worth considering for consistency.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We use "profile" a few times in these docs, but it's a technical detail that we've mostly avoided in the other docs. E.g. traces and logs both just use "experience".

Copy link
Copy Markdown
Member

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for working on this!

@natasha-moore-elastic natasha-moore-elastic merged commit f6930dd into main Mar 30, 2026
8 checks passed
@natasha-moore-elastic natasha-moore-elastic deleted the issue-3021-sec-profile branch March 30, 2026 13:03
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.

Create docs for Discover profile: Security

5 participants