Sam A.

Results 12 comments of Sam A.

@Cryptophobia I am having a similar problem with `mounted-file` as @vkadi. I believe the problem isn't in configMap changes, this part works as expected, reloader kicks in and invokes controller::RunOnce()...

> 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...

> @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...

> @sabdalla80, thank you for testing. After I do some local testing, I will build a new test image today. This would be spectacular. @Cryptophobia If you can share any...

> @Cryptophobia Are you able to see main control loop re-run when you restart/replace a pod in the deployment? Does it re-generate fluentd.conf file?

@Cryptophobia In intial deployment things work fine, the control loop generates fluentd.conf and proper CM files for a service. On pod restart, even though fluentd indeed sees the new pod,...

> @sabdalla80 , thank you for this. I think I understand the problem. The CW plugin requires that the container name be known ahead of time so even though we...

@Cryptophobia with `vmwaresaas.jfrog.io/vdp-public/upstream/kube-fluentd-operator:sabdalla80-test-1` I did see reloader log this event this time around on mounted-file pod restart. But, I did not see fluentd.conf file update. Are you not running the...

> @sabdalla80 I will wait for your test with container before I make PR: > > ``` > vmwaresaas.jfrog.io/vdp-public/upstream/kube-fluentd-operator:sabdalla80-test-2 > ``` @Cryptophobia I am still seeing the same behavior. It...

> @sabdalla80 , have you tried running KFO in `--datasource=crd` mode ? Seems to work for me in the crd mode. @Cryptophobia No, I have not tried. I suppose I...