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
Read configuration file with .yaml suffix #10909
Conversation
Fixes a bug whereby we failed to read an elasticsearch config file with the .yaml extension. This commit allows elasticsearch config files to be suffixed with: .yml, .yaml, .json, .properties. Closes #9706
Change looks good but I think we should have a test for it too. Also, I remember making the same change on |
One more thing, this can go into 1.x too right? |
I considered abstracting out the code from |
Yes, it can go into 1.x. |
fair enough, can we have a test for this change though? |
@javanna Added a test |
LGTM thanks @aleph-zero |
Added 1.6.0 and 1.5.3 labels, should be easy to backport ;) |
Read configuration file with .yaml suffix
Read configuration file with .yaml suffix
Read configuration file with .yaml suffix
Fixes a bug whereby we failed to read an elasticsearch config file with
the .yaml extension. This commit allows elasticsearch config files to
be suffixed with: .yml, .yaml, .json, .properties.
Closes #9706