-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Auditbeat] Add ECS categories for system-host module #18031
[Auditbeat] Add ECS categories for system-host module #18031
Conversation
Pinging @elastic/siem (Team:SIEM) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ECS categorization looks great.
Looks like we could populate the ECS host fields. Should we add that?
@leehinman so, the beats/x-pack/auditbeat/module/system/host/host.go Lines 118 to 125 in a601961
We currently populate:
Just for kicks, I added
|
💚 Build SucceededExpand to view the summary
Build stats
Test stats 🧪
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* [Auditbeat] Add ECS categories for system-host module * Add changelog entry * Add host.name * Add host.name to ECS-only fields * Remove added 'name' field (cherry picked from commit dece318)
What does this PR do?
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Related issues