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

[Auditbeat] Cherry-pick #12259 to 7.2: Host: Fill top-level host fields #12316

Merged
merged 1 commit into from
May 29, 2019

Conversation

cwurm
Copy link
Contributor

@cwurm cwurm commented May 28, 2019

Cherry-pick of PR #12259 to 7.2 branch. Original message:

As of now, the Auditbeat system/host dataset puts all its fields under system.audit.host.*, making its output hard to use together with the data usually added by the add_host_metadata processor in the top-level host object.

With this PR, the dataset copies its fields to host.* so those are always filled.

Most importantly, add_host_metadata does not fill host.ip and host.mac by default. They will now always be filled by this dataset.

I also added system.audit.host.os.codename to complete what the processor adds.

Copy fields of `system/host` dataset to `host.*` so those are always filled.

Also add `system.audit.host.os.codename` to complete what `add_host_metadata` adds.

(cherry picked from commit 5502f4e)
@cwurm cwurm requested review from a team as code owners May 28, 2019 19:42
@cwurm cwurm changed the title Cherry-pick #12259 to 7.2: [Auditbeat] Host: Fill top-level host fields [Auditbeat] Cherry-pick #12259 to 7.2: Host: Fill top-level host fields May 28, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/secops

@cwurm cwurm merged commit de51268 into elastic:7.2 May 29, 2019
@cwurm cwurm deleted the backport_12259_7.2 branch May 29, 2019 16:05
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…st fields (elastic#12316)

Copy fields of `system/host` dataset to `host.*` so those are always filled.

Also add `system.audit.host.os.codename` to complete what `add_host_metadata` adds.

(cherry picked from commit f6cebc1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants