AMO 🐶⛺🌳

+1

Hi @maxsivkov , thank you for the contributions! Looks good but I need to do a deeper review and test. I will merge this in at some point after we...

@vkadi , can you share the configMap or CRD fluentdconfig that you are using for the above testing container? Are you using type `@mounted-file`? Are you following the README here:...

Thank you @sabdalla80 , that description helps a lot! I think in this case maybe we are not doing it correctly. Since the config file has not change, we only...

I think this is related to the issues in #253

@sabdalla80 , yes I think we need to find some place in `kubeInformer.go` (config-reloader/datasource/kube_informer.go) where we can do this check via the sharedInformers for the pods. If pods are getting...

I just looked at the code and I have an idea. We can use an hasher to get a unique has for the nsConfig struct each time RunOnce() loop runs....

@sabdalla80, thank you for testing. After I do some local testing, I will build a new test image today.

@sabdalla80 , When I do a simple test with 3 pods in a deployment. I cannot reproduce this. You are saying the mountedfile:Process() is not being called. Can you be...

No, the main control loop does not rerun. But I see fluentd picks up new pods and new container names when the containers restarts/scale-up or whatever. This seems like a...