You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Jul 11, 2023. It is now read-only.
Please describe the Improvement and/or Feature Request
We currently tightly couple our informers to specific objects (e.g. k8s controller, configurator, smi, and policy). The drawback to this approach is that no other component of the code can access these informer caches, respond to events, or otherwise interact with k8s objects except by:
depending on one of the aforementioned objects (creating coupling where it may not need to be)
creating new informers and duplicating the event handling/caching (opening the door for inconsistencies between the two caches)
I'm proposing that we centralize our informer handling in one object whose pointer can be passed around to any other object that may need to access an informer. This prevents engineers from having to create new, expensive informers, and it allows us to have opinionated patterns for informer use.
Scope (please mark with X where applicable)
Envoy Control Plane [X]
Possible use cases
The text was updated successfully, but these errors were encountered:
Please describe the Improvement and/or Feature Request
We currently tightly couple our informers to specific objects (e.g. k8s controller, configurator, smi, and policy). The drawback to this approach is that no other component of the code can access these informer caches, respond to events, or otherwise interact with k8s objects except by:
I'm proposing that we centralize our informer handling in one object whose pointer can be passed around to any other object that may need to access an informer. This prevents engineers from having to create new, expensive informers, and it allows us to have opinionated patterns for informer use.
Scope (please mark with X where applicable)
Possible use cases
The text was updated successfully, but these errors were encountered: