Stefan Prodan

Results 832 comments of Stefan Prodan

Is this a transient error or the backend is never added?

I can't replicate this in CI. Do you see anything else in the logs? If you restart the gateway does it add it?

So i did some digging into Helm SDK source code. The culprit seems to be the [getCapabilities](https://github.com/helm/helm/blob/14d0c13e9eefff5b4a1b511cf50643529692ec94/pkg/action/action.go#L246) which invalidates the client CRD cache and then queries the Kubernetes API to...

Some good new 🎉 A combination of improvements in Flux 2.3 and Kubernetes API 1.29/1.30 make this issue less impactful. Compared to Flux 2.2 and Kubernetes 1.28 where large number...

@seaneagan I don't see how `.Status.ObservedGeneration` is not updated if the release upgrade succeeds given: https://github.com/fluxcd/helm-controller/blob/5e760db4a83ffd384012c1b47e4615cc07d4081d/internal/controller/helmrelease_controller.go#L180-L182 Can you please add a unit test to prove this can actually happen? Removing...

I think we need a test for when drift correction is in loop that should used the kstatus verifier that catches any miss configuration https://github.com/fluxcd/kustomize-controller/blob/e9f5628eccbfbc722a7637ecbf7f66580e2e4416/internal/controller/kustomization_wait_test.go#L135 The checks and logic we...

I think this will fix it https://github.com/fluxcd/helm-controller/pull/885

@seaneagan with #885 merged I think this will solve your issue. Ready will reflect now that drift detection is running without reaching correction.

> We need a test case for idempotence here. Reading this I hope we are safe: - https://github.com/helm/helm/blob/main/pkg/chartutil/dependencies.go#L208-L211 - https://github.com/helm/helm/blob/main/pkg/chartutil/dependencies.go#L303-L315 In any case, if anywhere in that logic a map...

> The bootstrap command only works with setting up a new project (that doesn't already exist). Never heard this before from GitLab users, Flux bootstrap should detect if a repo...