Skip to content

[6.x] Update beats framework to c63c98b #1647

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 13, 2018
Merged

[6.x] Update beats framework to c63c98b #1647

merged 1 commit into from
Dec 13, 2018

Conversation

@graphaelli
Copy link
Member

@graphaelli graphaelli commented Dec 12, 2018

  • adds aliases for 7.x beats fields

@graphaelli
Copy link
Member Author

graphaelli commented Dec 12, 2018

Only the sync job failed, as expected since this change adds new alias fields:

cloud.availability_zone
cloud.instance.id
cloud.instance.name
cloud.machine.type
cloud.project.id
cloud.provider
cloud.region
container.id
container.image.name
container.name
event.timezone
host.hostname

This was referenced Dec 12, 2018
Merged
Merged
@graphaelli graphaelli changed the title Update beats framework to c63c98b [6.x] Update beats framework to c63c98b Dec 13, 2018
simitt
simitt reviewed Dec 13, 2018
View reviewed changes
vendor/vendor.json
"revisionTime": "2018-11-21T22:04:10Z",
"version": "release-branch.go1.10",
"versionExact": "release-branch.go1.10"
},
Copy link
Contributor

@simitt simitt Dec 13, 2018

Choose a reason for hiding this comment

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

Out of curiosity - do you know why those were removed now?

Copy link
Member Author

@graphaelli graphaelli Dec 13, 2018

Choose a reason for hiding this comment

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

That's on me - they should have not made it in with the backport in #1587

@graphaelli graphaelli merged commit 177a7d2 into elastic:6.x Dec 13, 2018
@graphaelli graphaelli deleted the update-beats-6.x branch December 13, 2018 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@simitt simitt simitt approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@graphaelli @simitt