Skip to content

Cherry-pick #9408 to 6.x: Add event.dataset field#9561

Merged
andrewvc merged 3 commits intoelastic:6.xfrom
andrewvc:backport_9408_6.x
Dec 20, 2018
Merged

Cherry-pick #9408 to 6.x: Add event.dataset field#9561
andrewvc merged 3 commits intoelastic:6.xfrom
andrewvc:backport_9408_6.x

Conversation

@andrewvc
Copy link
Copy Markdown
Contributor

Cherry-pick of PR #9408 to 6.x branch. Original message:

This brings us closer to ECS compliance by adding the event.dataset field.

All heartbeat events are considered part of the 'uptime' dataset, even though their data may differ.

The key thing defining this dataset is the presence of the monitor.status field.

This brings us closer to ECS compliance by adding the event.dataset field.

All heartbeat events are considered part of the 'uptime' dataset, even though their data may differ.

The key thing defining this dataset is the presence of the `monitor.status` field

(cherry picked from commit 5e54acb)
@ruflin
Copy link
Copy Markdown
Contributor

ruflin commented Dec 17, 2018

The build fails because event.dataset is not document. It is globally document in 7.0 but in 6.x each Beat that uses is needs to document it. Adding it to heartbeat/_meta/fields.common.yml should fix the issue.

@ruflin
Copy link
Copy Markdown
Contributor

ruflin commented Dec 18, 2018

jenkins, test this

@andrewvc
Copy link
Copy Markdown
Contributor Author

@ruflin are you good with me merging this? Jenkins is passing, travis is not but it looks unrelated

@andrewvc andrewvc merged commit 5a30d96 into elastic:6.x Dec 20, 2018
@andrewvc andrewvc added v6.6.0 and removed v6.6.0 labels Jan 3, 2019
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.

2 participants