argoproj / gitops-engine

Democratizing GitOps
https://pkg.go.dev/github.com/argoproj/gitops-engine?tab=subdirectories
Apache License 2.0
1.67k stars 251 forks source link

fix: Deadlock issue caused in listResources (argoproj/argo-cd#18902) #599

Closed jdroot closed 1 month ago

jdroot commented 2 months ago

Checklist:

Fixes argoproj/argo-cd#18902

agaudreault commented 1 month ago

Superseded by https://github.com/argoproj/gitops-engine/pull/604. If memory consumption is too unpredictible with the change, let revisit and check if we allow a mechanism to skip the semaphore 👍 Good job on finding the deadlock nontheless, I wish I had saw your issue sooner!