fluent / fluent/helm-charts

[fluentd] Add separate aggregator chart

Open
#67 12 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Go Template
Stars
473
Forks
497
Avg merge
22h 8m
Merged PRs (30d)
3

Description

The two use cases for _Fluentd_ are very different in implementation and as such rather than trying to shoehorn both into a single chart it would make sense to have a separate `fluentd-aggregator` chart. I already have a [fluentd-aggregator](https://github.com/stevehipwell/helm-charts/tree/master/charts/fluentd-aggregator) chart in my repo and would be happy to create a PR to add it to this repo and deprecate mine.

It would also be useful to have an aggregator _Docker_ image to handle the common aggregation endpoints; an aggregator isn't as concerned with size and if the plugins are layered on top of the base image it would keep the layers easily portable. I've created my own [image](https://github.com/stevehipwell/fluentd-aggregator) for my _Helm_ chart, although the chart will work fine with the base _Fluentd_ image (by default it outputs to null and has stdout debugging available), but an official version would be better.

I'd like to call out @MrD2 for his work on PR https://github.com/fluent/helm-charts/pull/55 for the _Fluentd_ collector chart, this would be intended to augment this not replace it.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.