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

Added migration script from 1.x to 5.0 configuration files #1576

Merged
merged 1 commit into from
May 6, 2016

Conversation

tsg
Copy link
Contributor

@tsg tsg commented May 4, 2016

It doesn't do yaml parsing, just line by line string operations. This means
it is not able to transform any yaml syntax, but should work on the most
common cases.

On the plus side, it uses only python standard library modules and it keeps
the comments as in the original file. Related to #1488.

It doesn't do yaml parsing, just line by line string operations. This means
it is not able to transform any yaml syntax, but should work on the most
common cases.

On the plus side, it uses only python standard library modules and it keeps
the comments as in the original file. Related to elastic#1488.
@tsg tsg added discuss Issue needs further discussion. review labels May 4, 2016
@ruflin
Copy link
Collaborator

ruflin commented May 4, 2016

I really like the idea of having this kind of scripts. Alternatively it could be shipped as an additional Golang binary (beat-helper ;-) ) but as we know, YAML parsing etc. has its issues here. Python is probably available on most environments and it is cross platform compatible.

@andrewkroh andrewkroh merged commit be4beb3 into elastic:master May 6, 2016
@tsg tsg deleted the migrate_script branch August 25, 2016 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Issue needs further discussion. review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants