operator-framework / operator-controller

Apache License 2.0
28 stars 47 forks source link

🐛 fix: reduce unnecessary caching and remove predicate from dynamic watches #963

Closed joelanford closed 1 week ago

joelanford commented 1 week ago

Our cache configuration resulted in watching all cluster-scoped resources for each cluster-scoped resource that shows up in a ClusterExtension. Now, we only watch:

This commit also removes the dependent predicate so that we reconcile for any change to managed resources. This is important to ensure that future features (like health checking) can account for any change to managed operator objects.

Description

Reviewer Checklist

netlify[bot] commented 1 week ago

Deploy Preview for olmv1 ready!

Name Link
Latest commit 65aa63c455521e26699a006ba52a28279ae9c7e3
Latest deploy log https://app.netlify.com/sites/olmv1/deploys/66759d3e0bbda100085de84b
Deploy Preview https://deploy-preview-963--olmv1.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.