keda icon indicating copy to clipboard operation
keda copied to clipboard

Fix scaler leak during cache refresh

Open gjacquet opened this issue 3 months ago • 5 comments

Fixes scaler not being properly closed in performGetScalerCache:

  • Fix logic to close scaler when out of date
  • Fix race confition

Checklist

  • [x] When introducing a new scaler, I agree with the scaling governance policy
  • [x] I have verified that my change is according to the deprecations & breaking changes policy
  • [ ] Tests have been added
  • [x] Changelog has been updated and is aligned with our changelog requirements
  • [ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
  • [ ] A PR is opened to update the documentation on (repo) (if applicable)
  • [x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #5806

gjacquet avatar May 15 '24 14:05 gjacquet

/run-e2e Update: You can check the progress here

zroubalik avatar May 17 '24 10:05 zroubalik

@gjacquet can you confirm that this fix helped with the issues in your env?

zroubalik avatar May 17 '24 10:05 zroubalik

@zroubalik Thanks for reviewing! I have been running this patch in two of our non-production environments and it solved this issue.

gjacquet avatar May 17 '24 11:05 gjacquet

Thanks, this is really great fix

zroubalik avatar May 17 '24 11:05 zroubalik

Awesome catch! Thanks for the fix :heart:

JorTurFer avatar May 18 '24 09:05 JorTurFer