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
I'm going to use this issue to keep track of some conversations and clean up tasks that are being generated out of the sync set proposal implementation.
with the CacheManager from feat: support multiple sync sources #2852, a lot of the dep Injectors for config_c are redundant. Let's take a pass and clean them all up
AuditFromCache lister requires us to keep a watchSet in the dep for now. Let's revisit that decision. Audit could call out to CM somehow to get the watchSet or answer the question of "what is in the informer cache". implemented as part of feat: support multiple sync sources #2852
Tasks to think about:
need to update readiness tracker code to deal with sync sets when adding controller
I'm going to use this issue to keep track of some conversations and clean up tasks that are being generated out of the sync set proposal implementation.
Tasks deferred from #2852:
Refactors:
with theCacheManager
from feat: support multiple sync sources #2852, a lot of the dep Injectors for config_c are redundant. Let's take a pass and clean them all upindeed, many of the adder's funcs aren't even implemented anymore; can simplify the interfaces maybedone as part of refactor: adder interface, rename data client #2991implemented as part of feat: support multiple sync sources #2852AuditFromCache
lister requires us to keep awatchSet
in thedep
for now. Let's revisit that decision. Audit could call out to CM somehow to get thewatchSet
or answer the question of "what is in the informer cache".Tasks to think about:
ifdone as part of chore: require-sync gk update gatekeeper-library#396parser
package is moved, this might need an update feat: use gatekeeper parser for require-sync gatekeeper-library#367The text was updated successfully, but these errors were encountered: