chore: Advance libsveltos pin and drift-detection-manager image tag - #1916
Merged
Conversation
This will pick: 1. An optimization on the CustomResourceDefinition watcher: replaces the SharedIndexInformer used to watch CRDs with a lean client-go Reflector that forwards each Add/Update/Delete straight to the handler and discards the object, instead of retaining every CRD in an indexed cache for the process lifetime. Reduces memory consumption on every Sveltos component, including addon-controller, starting that watcher. 2. The same optimization applied to drift-detection-manager's per-GVK resource watchers: replaces the DynamicSharedInformerFactory backed informer used to watch tracked resources with the same reflector/discarding-store approach. Nothing there ever read the cached store back (drift checks always re fetch the resource live to evaluate if a configuration drift happened), so it was pure overhead, retained per watched GVK for as long as the watcher stayed active. Reduces drift-detection-manager memory on clusters tracking many resources across many GVKs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This will pick: