Skip to content

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

Merged

Conversation

smarterclayton
Copy link
Contributor

@smarterclayton smarterclayton commented Jun 10, 2017

Also make it available to GC. Remove "images" from GC for now.

Only the last two commits are new to this PR.

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.
@smarterclayton smarterclayton force-pushed the remove_imagestream_legacy branch from 00da5b6 to c42f8b0 Compare June 10, 2017 04:43
@smarterclayton
Copy link
Contributor Author

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 78917fc

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/2091/) (Base Commit: c9a1d10)

@smarterclayton
Copy link
Contributor Author

Build flake

@smarterclayton
Copy link
Contributor Author

@mfojtik this is part of a chain, please review image stream generator changes

@mfojtik
Copy link
Contributor

mfojtik commented Jun 11, 2017

@smarterclayton LGTM good stuff. this will probably break you: #14317 (or you break me 👿 )

@smarterclayton
Copy link
Contributor Author

[merge] but I bet you beat me in

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 78917fc

@openshift-bot
Copy link
Contributor

openshift-bot commented Jun 12, 2017

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)

@openshift-bot openshift-bot merged commit aed6393 into openshift:master Jun 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants