search

kube-logging / telemetry-controller

A multi-tenancy focused solution, that facilitates collection of telemetry data from Kubernetes workloads transparently.
Apache License 2.0
29 stars 0 forks source link

We are logging errors on the first reconciliation #63

Closed kristofgyuracz closed 3 weeks ago

kristofgyuracz commented 7 months ago

When the two controllers (route and collector) are updating the tenant status, we are logging reconciler errors, because the resource being updated was changed in the meantime by the other controller. We sould have some kind of locking to prevent this.

pepov commented 7 months ago

it could perhaps be easier to just return instantly when writing the status. it will then be two reconciliation to reach the final state, but we could avoid errors because of these updates

pepov commented 3 months ago

is this still relevant?

kristofgyuracz commented 2 months ago

I think it's still present.