-
Notifications
You must be signed in to change notification settings - Fork 594
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"flux create source helm" randomly fails reconciliation with "no artifact was found" #2736
Comments
Hey @artem-nefedov , |
Hi @somtochiama You can either tell me what debug info I can collect, or leave/close the ticket until someone else has the same problem. |
Can you see any logs from the source-controller? |
Looking at https://github.com/fluxcd/flux2/blob/main/cmd/flux/create_source_helm.go#L199, it seems there was really no artifact in helm repository. Maybe the helm chart isn't ready yet and some seconds later (when you run the second command) it is. |
That could be the case, but then it begs the question: why can source-controller report that reconcile has finished before artifact is ready? It looks like there's a race condition in play here. Or, if that's a normal behaviour for the controller, why doesn't flux cli wait for artifact to appear? For now, I've added a workaround to immediately call
And, from what I see, the reconcile always succeeds. |
This should've been fixed in Flux v2.2 |
Describe the bug
flux create source helm
command would occasionally fail with an error:The actual HelmRepository object is fine and usable. If you immediately call
flux reconcile source helm
on it after encountering that error, it will succeed.Steps to reproduce
flux create source helm
Expected behavior
If there's a race condition, flux cli must properly wait for artifact to appear.
Screenshots and recordings
No response
OS / Distro
Linux
Flux version
v0.30.2
Flux check
fluxcd/helm-controller:v0.21.0
fluxcd/source-controller:v0.24.4
Git provider
No response
Container Registry provider
No response
Additional context
We've never seen this problem on Flux v0.28.5. After upgrading to Flux v0.30.2, we now see it regularly.
Code of Conduct
The text was updated successfully, but these errors were encountered: