helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

[fluentd] Allow disable defaults

Open MioOgbeni opened this issue 3 years ago • 1 comments

Hi, I want to use FluentD but only for forwarded logs/metrics from FluentBit. So I don't want to use and have the default configuration for systemd and prometheus deployed in my K8s (my FluentD will consume data only by forward input plugin).

I made some changes that will allow to disable deploying of default configuration manifest an also keeps original functionality from default values.yaml.

  • [x] Fixed datatype of dashboards.enabled parameter
  • [x] Hardcoded configmaps fluentd-prometheus-conf and fluentd-systemd-conf can be now disabled
  • [x] fileConfigs parameter changed from map to array (allows you to insert different configuration fields here, but exclude the original one hardcoded)

Can you review it and tell if here is something wrong or out of yours vision? @edsiper @dioguerra

MioOgbeni avatar Mar 23 '23 10:03 MioOgbeni

I am also interested in this use-case snce I am using fluentd to collect only application logs. @MioOgbeni - are you following up with this?

@dioguerra - should I open a new PR to add this functionality?

uristernik avatar Dec 31 '23 08:12 uristernik