Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[aws-for-fluent-bit] Is this chart the recommended way to setup FluentBit on AWS EKS? #719

Open
trallnag opened this issue Mar 24, 2022 · 3 comments · May be fixed by #1168
Open

[aws-for-fluent-bit] Is this chart the recommended way to setup FluentBit on AWS EKS? #719

trallnag opened this issue Mar 24, 2022 · 3 comments · May be fixed by #1168

Comments

@trallnag
Copy link

Asking because the AWS documentation does not mention this chart anywhere.

@trallnag trallnag changed the title [aws-for-fluent-bit] Is this chart the recommended way to setup Fluent Bit on AWS EKS? [aws-for-fluent-bit] Is this chart the recommended way to setup FluentBit on AWS EKS? Mar 24, 2022
@matti
Copy link

matti commented Aug 1, 2022

related aws/aws-for-fluent-bit#357

@mengmann
Copy link

mengmann commented Oct 14, 2022

We are not only dealing with an inconsistency in the way the FluentBit installation is supposed to be done according to the official AWS documentation (custom kubectl manifests vs. helm chart). Rather, we are also dealing with a difference in content. This can be seen just by looking at the ConfigMap for Cloudwatch Logs integration.

The referenced kubectl manifests (this and this) from AWS ensures that the logs are sorted into different LogGroups depending on their origin / meaning: application, host, dataplane. This is done via different include files inside the fluent-bit.conf. In the default version of the ConfigMap from the Helm chart, only a simple vanilla config is generated.

So why do we have different default settings here? How can the helm chart ConfigMap be overridden to look like the "official" variant regarding Cloudwatch Logs? This seems currently not possible because the current configmap template is very limited.

@joebowbeer
Copy link
Contributor

FWIW, we've opted to use the fluent-bit helm chart from https://fluent.github.io/helm-charts

and then patch in the image provided by AWS: public.ecr.aws/aws-observability/aws-for-fluent-bit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants