-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Replace legacy IS informer with generated informers everywhere #14562
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace legacy IS informer with generated informers everywhere #14562
Conversation
Allow them to be loaded via ForResource() as a stepping stone to transforming their behavior. Set projects and tokens as GC excluded resources (virtual, and security related).
Also make it available to GC. Remove "images" from GC for now.
00da5b6
to
c42f8b0
Compare
[test] |
Evaluated for origin test up to 78917fc |
continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/2091/) (Base Commit: c9a1d10) |
Build flake |
@mfojtik this is part of a chain, please review image stream generator changes |
@smarterclayton LGTM good stuff. this will probably break you: #14317 (or you break me 👿 ) |
[merge] but I bet you beat me in |
Evaluated for origin merge up to 78917fc |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/968/) (Base Commit: e166740) (Image: devenv-rhel7_6338) |
Also make it available to GC. Remove "images" from GC for now.
Only the last two commits are new to this PR.