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

Remove convert old states code #3767

Merged
merged 1 commit into from
Mar 22, 2017
Merged

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Mar 17, 2017

Filebeat 1.x had a different registry format from 5.x. So a conversion of the format happened. This code is now removed for 6.0 as the upgrade path to 6.0 is from 5.0.

The code caused some issues in the past so it is better to not have it in anymore.

@ruflin ruflin added Filebeat Filebeat in progress Pull request is currently in progress. labels Mar 17, 2017
@@ -20,6 +20,7 @@ https://github.com/elastic/beats/compare/v5.1.1...master[Check the HEAD diff]

*Filebeat*
- Always use absolute path for event and registry. This can lead to issues when relative paths were used before. {pull}3328[3328]
- Remove code to convert states from 1.x.
Copy link
Contributor

Choose a reason for hiding this comment

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

Link to PR please.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, still has in in progress, nevermind :)

Copy link
Member Author

Choose a reason for hiding this comment

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

good you mentioned it, would have forgotten otherwise ;-)

Filebeat 1.x had a different registry format from 5.x. So a conversion of the format happened. This code is now removed for 6.0 as the upgrade path to 6.0 is from 5.0.

The code caused some issues in the past so it is better to not have it in anymore.
@ruflin ruflin force-pushed the remove-convert-old-states branch from f2a7f07 to 91fa70d Compare March 22, 2017 11:50
@ruflin ruflin added review and removed in progress Pull request is currently in progress. labels Mar 22, 2017
@tsg tsg merged commit fc15b80 into elastic:master Mar 22, 2017
@ruflin ruflin deleted the remove-convert-old-states branch May 10, 2017 10:25
@tsg tsg mentioned this pull request Jul 24, 2017
28 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants