[Workflows] Event-driven triggers GA in 9.5 and entity store triggers - #7568
[Workflows] Event-driven triggers GA in 9.5 and entity store triggers#7568natasha-moore-elastic wants to merge 4 commits into
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
Elastic Docs AI PR menuCheck the box to run an AI review for this pull request.
Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team. |
✅ Elastic Docs Style Checker (Vale)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. |
ymao1
left a comment
There was a problem hiding this comment.
LGTM for entity store trigger changes
|
|
||
| | Field | Contains | | ||
| |---|---| | ||
| | `event.entityId` | The unique identifier (EUID) of the entity whose asset criticality changed. | |
There was a problem hiding this comment.
also a nit: EUID = Entity Unique ID I'm guessing :D This is not the common meaning of this acronym, but maybe it's obvious enough like this, or maybe move (EUID) to just after "entity"?
There was a problem hiding this comment.
I've tried placing it after entity, but I feel that makes it sound like the acronym refers to the entity, not the identifier, so I'd prefer to keep it as is.
Co-authored-by: Florent LB <florent.leborgne@elastic.co>
| | Field | Contains | | ||
| |---|---| | ||
| | `event.entityId` | The unique identifier (EUID) of the entity whose asset criticality changed. | | ||
| | `event.entityType` | The entity type (`host`, `user`, or `service`). | |
There was a problem hiding this comment.
generic is missing here, is it intentional?
There was a problem hiding this comment.
Hey @yngrdyn, good question. I recently discussed the availability of the generic entity type with @CAWilson94 when I was working on a different Entity Analytics docs issue. We agreed to generally not include it in the docs since it's gated behind a flag that's off by default, and isn't exposed in the Entity Analytics UI. But if we should treat these docs differently, let me know.
cc @ymao1
There was a problem hiding this comment.
I think it's fine to exclude here given your discussion with Charlotte. I'm not sure what the future of generic is so we can add it to the docs in the future if needed
| @@ -343,13 +341,126 @@ triggers: | |||
| condition: 'event.owner: "securitySolution"' | |||
| ``` | |||
|
|
|||
There was a problem hiding this comment.
You'll get an extra blank line here, I don't know if this will affect the rendering of this .md file
There was a problem hiding this comment.
Hey, not sure I understand – the blank line is just between the code example and the next section. You can see a preview of how it renders here, LMK if anything looks off.
Summary
Updates the Workflows event-driven triggers docs for the 9.5 release:
Resolves elastic/docs-content-internal#1449
Related docs PRs
The cases and alerting-system trigger families were already documented in earlier PRs, so they aren't part of this PR: