-
Notifications
You must be signed in to change notification settings - Fork 852
Validate configuration #64
Comments
@babadofar whilst i like the idea of this feature, any validation will require us knowledge of the settings. This will somewhat negate the purpose of the map. However, if it is just syntax checks you are after that maybe possible? |
When using the template function of ansible, there is an option to make it validate the file that results from running the template using a command that you can specify. I have used this previously to validate Logstash configurations, using the Logstash --configtest parameter. I'm not sure whether there is such a parameter to elasticsearch. So perhaps this issue is better moved there? |
@babadofar yes this is indeed possible in ES. I'll investigate and see if we can do this with ES. |
@babadofar So configuration validation is coming in ES 5.0. I will raise a ticket asking for a -t flag and reference |
Discussing with ES dev the best way to do this will be to start a standalone node with the configuration. I'm looking at ways to achieve this prior to running the playbook as a pre-check. |
Docker may be a nice solution for this.
Assuming that running |
@strootman i was thinking we would need to download the version the user is using - as config is subject to change. Effectively local-action commands. This way we can warn if settings are deprecated etc. This is likely to be a completely separate task with an optional switch. Happy for someone to take this on as i'll be focusing on 5.0 support after the xpack pull request. |
This issue has been automatically marked as stale because it has not had |
This issue has been automatically closed because it has not had recent activity since being marked as stale. |
@gingerwizard Was Configuration validation ever released? |
Any news on this one? |
Any news under the sun? |
Using a map for configuring elasticsearch.yml is super flexible and nice, but it makes it very easy to miscreate configurations that will not start. It would be very nice if there was some sort of validation that checks syntax.
The text was updated successfully, but these errors were encountered: