kiwigrid / helm-charts

Helm charts for Kubernetes curated by Kiwigrid
https://kiwigrid.github.io
MIT License
186 stars 209 forks source link

[fluentd-elasticsearch] Allow multiple hosts in config #330

Closed samuel-begin closed 4 years ago

samuel-begin commented 4 years ago

FEATURE REQUEST

Since Fluentd config accepts multiple hosts in its config, I would like to be able to provide them through the chart.

So migrating the parameter: Parameter Description Default
elasticsearch.host Elasticsearch Host elasticsearch-client
to: Parameter Description Default
elasticsearch.hosts[] Elasticsearch Hosts list []
samuel-begin commented 4 years ago

I'll be working on this, since we need it for our deployment

Would you prefer to deprecate host and have migration doc? or you'd rather stay backwards compatible and have both parameters host and hosts[]?

monotek commented 4 years ago

I would prefer to deprecate host and have migration doc.