search

kiwigrid / helm-charts

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

Filter out fluentd logs #367

Closed jopfeffe closed 4 years ago

jopfeffe commented 4 years ago

Is this a BUG REPORT or FEATURE REQUEST? (choose one): FEATURE REQUEST

Version of Helm and Kubernetes: 3.2.4 + K8S 1.17

Which chart in which version: fluentd-elasticsearch 9.3.2

What happened: Fluentd Logs are shipped to elastic search.

What you expected to happen: At least there should be a configuration option to disable this behavior.

<match kubernetes.var.log.containers.{{ include "fluentd-elasticsearch.fullname" . }}**.log>
      @type null
</match>

Kind regards Jochen

monotek commented 4 years ago

you already can do this:

https://github.com/kiwigrid/helm-charts/blob/master/charts/fluentd-elasticsearch/values.yaml#L236-L250

jopfeffe commented 4 years ago

I know that I can replace the entire config. But I thought it's a little bit overkill for this common config.

monotek commented 4 years ago

pr welcome ;-)

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.