search

kiwigrid / helm-charts

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

[fluentd-elasticsearch] change logstash_dateformat to a weekly format #373

Closed rgarcia89 closed 4 years ago

rgarcia89 commented 4 years ago

I would like to switch from daily to weekly indexes. Therefore I tried to pass the env variable logstash_dateformat

For testing I first tried the following

env:
  LOGSTASH_DATEFORMAT: "%Y"

I can see the env being set. However, the index still is getting yyyy.mm.dd as suffix. According to https://www.rubydoc.info/gems/fluent-plugin-elasticsearch/1.5.0#logstash_dateformat this variable should bring the change. Am I missing something?

I am happy to create a pull request so that this variable can be set via the values file, once we have figured out, why it is not working.

Thanks in advance

rgarcia89 commented 4 years ago

I have figured out the reason and created a pull request https://github.com/kiwigrid/helm-charts/pull/374

Looking forward to get merged Thanks

rgarcia89 commented 4 years ago

New pull request: https://github.com/kiwigrid/helm-charts/pull/375

monotek commented 4 years ago

merged...