Skip to content
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

UPSTREAM: 29588: Init container quota is inaccurate #10028

Merged

Conversation

smarterclayton
Copy link
Contributor

@derekwaynecarr but please review upstream first.

[test]

@smarterclayton
Copy link
Contributor Author

[test] #9490

On Tue, Jul 26, 2016 at 4:00 AM, OpenShift Bot notifications@github.com
wrote:

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


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#10028 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABG_pxkLWiYA5p36F1ersaqff9aYP8gJks5qZb6xgaJpZM4JUuK5
.

@deads2k
Copy link
Contributor

deads2k commented Jul 26, 2016

@smarterclayton make sure it works with clusterquota. I think that we should get it for free, but I'd like to be sure.

@smarterclayton
Copy link
Contributor Author

I checked but I'll add it to the cluster quota tests as well.

On Tue, Jul 26, 2016 at 9:45 AM, David Eads notifications@github.com
wrote:

@smarterclayton https://github.com/smarterclayton make sure it works
with clusterquota. I think that we should get it for free, but I'd like to
be sure.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#10028 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABG_p5WtV4NGK2w7WmdUj1cgZfmUn2rLks5qZg-HgaJpZM4JUuK5
.

@smarterclayton
Copy link
Contributor Author

Found a bug in this, waiting to merge upstream then will fix.

On Tue, Jul 26, 2016 at 11:45 AM, OpenShift Bot notifications@github.com
wrote:

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


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#10028 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABG_pzrr7Cx-O0neIlWfNSTcrfqXSfRDks5qZiuRgaJpZM4JUuK5
.

@smarterclayton
Copy link
Contributor Author

Updated based on merged upstream PR

@stevekuznetsov
Copy link
Contributor

re[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to ba33f9a

@openshift-bot
Copy link
Contributor

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

@smarterclayton
Copy link
Contributor Author

smarterclayton commented Aug 1, 2016 via email

@openshift-bot
Copy link
Contributor

openshift-bot commented Aug 1, 2016

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

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to ba33f9a

@openshift-bot openshift-bot merged commit b729330 into openshift:master Aug 2, 2016
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.

None yet

4 participants