Skip to content

Imagestream limits #10517

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
merged 3 commits into from
Aug 19, 2016
Merged

Imagestream limits #10517

merged 3 commits into from
Aug 19, 2016

Conversation

deads2k
Copy link
Contributor

@deads2k deads2k commented Aug 18, 2016

Prevents bypassing imagestream count limits. Found a couple different bugs causing this.

@mfojtik or @miminar ptal

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1368078

@deads2k
Copy link
Contributor Author

deads2k commented Aug 18, 2016

[test]

@mfojtik
Copy link
Contributor

mfojtik commented Aug 18, 2016

@deads2k rocks! thanks!

LGTM

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to a391266

InternalGroupKind: imageapi.Kind("ImageStreamTag"),
InternalOperationResources: map[admission.Operation][]kapi.ResourceName{
admission.Update: computeResources,
admission.Create: computeResources,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So that's why! I forgot you added istag.Create() recently.

@miminar
Copy link

miminar commented Aug 18, 2016

LGTM

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/8166/)

@deads2k
Copy link
Contributor Author

deads2k commented Aug 18, 2016

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Aug 18, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/8166/) (Image: devenv-rhel7_4871)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to a391266

@openshift-bot openshift-bot merged commit d970aba into openshift:master Aug 19, 2016
@deads2k deads2k deleted the imagestream-limits branch September 6, 2016 17:15
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.

4 participants