Pat

Results 478 comments of Pat

@pmcenery please resolve the linting failure - either ignore with justification or correct.

We need to resolve conflicts before anything will be possible.

I'm not seeing this error locally with a fresh cluster (KIND) and a helm install: ``` $ kind create cluster Creating cluster "kind" ... ✓ Ensuring node image (kindest/node:v1.31.0) 🖼...

I think you have an older installation of the operator in your cluster in which case the docs show how to replace the CRDs for it: https://github.com/fluent/fluent-operator?tab=readme-ov-file#deploy-fluent-operator-with-helm You do have...

Can you sort the DCO check please? That's required to merge.

@rfloyrac can you sort the DCO? It's a hard requirement

> Can anyone review this? Did you update from the previous review?

Please resolve the DCO failure as well - that's required in order to merge.

https://docs.fluentbit.io/manual/administration/buffering-and-storage should cover it. It does depend specifically on your data rates/size plus actual config (e.g. plugins used/filters/etc.) but hopefully the docs give you that info. Also you're on an...