-
Notifications
You must be signed in to change notification settings - Fork 527
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
[6.6] Remove ECS aliases #1795
[6.6] Remove ECS aliases #1795
Conversation
jenkins, retest this please |
1 similar comment
jenkins, retest this please |
- from: beat.hostname | ||
to: host.hostname | ||
alias6: true | ||
alias: true |
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.
Why are we aliasing this field and not including it in the reindexing script? And shouldn't beat.hostname
go to observer.hostname
?
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.
Sorry, this is confusing. This doesn't create an alias, it's just beats' log of their aliases. The only aliases that are actually created are listed in the description.
* remove APM ECS comptability aliases * Revert "don't migrate beat.hostname" This reverts commit 8d8b859. * Revert "automate conflicting libbeat field removals (elastic#1665)" This reverts commit 02a5147. * revert fmt update
reverts most of #1609, part of #1670, and all of #1665.
aliases that remain are inherited from libbeat processors:
closes #1794
will need a backport to 6.x