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

Revert back to using host config option for defining multiple hosts #67

Closed
suyograo opened this issue Mar 4, 2015 · 3 comments
Closed
Assignees

Comments

@suyograo
Copy link
Contributor

suyograo commented Mar 4, 2015

1.4.2 has config option host http://logstash.net/docs/1.4.2/outputs/elasticsearch#host

this was changed in logstash-plugins/logstash-input-elasticsearch@add0954 which is not backward compatible

We need to go back to using host (even if it supports multiple hosts now)

@jordansissel
Copy link
Contributor

Yep! :validate => :arrray allows single-value settings (intentionally, yay config parser!) so host => "foo" and host => [ "foo" ] are equivalent in logstash config.

+1 for going back to host

@suyograo
Copy link
Contributor Author

suyograo commented Mar 4, 2015

@suyograo suyograo closed this as completed Mar 4, 2015
@suyograo
Copy link
Contributor Author

suyograo commented Mar 4, 2015

Tested fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants