Skip to content

Add quota controller metrics #10307

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 2 commits into from
Aug 10, 2016
Merged

Conversation

deads2k
Copy link
Contributor

@deads2k deads2k commented Aug 9, 2016

Adds metrics to the clusterquota controller so we can see what's running slow in the HA env.

@liggitt can you help me with magic incantation to get a dev ami?

@liggitt
Copy link
Contributor

liggitt commented Aug 9, 2016

@deads2k
Copy link
Contributor Author

deads2k commented Aug 9, 2016

[test]

@deads2k
Copy link
Contributor Author

deads2k commented Aug 9, 2016

re[test]

@deads2k
Copy link
Contributor Author

deads2k commented Aug 9, 2016

@danmcp @stevekuznetsov forkami ran out of memory: https://ci.openshift.redhat.com/jenkins/job/fork_ami/273/parameters/

FAIL    github.com/openshift/origin/pkg/cmd/server/kubernetes [build failed]
# testmain
/usr/lib/golang/pkg/tool/linux_amd64/link: running gcc failed: fork/exec /bin/gcc: cannot allocate memory

@danmcp
Copy link
Contributor

danmcp commented Aug 9, 2016

@deads2k bumped instance size and setup ram disk since it hadn't been added there yet: https://ci.openshift.redhat.com/jenkins/job/fork_ami/274/

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 3e83a4c

@openshift-bot
Copy link
Contributor

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

@deads2k
Copy link
Contributor Author

deads2k commented Aug 10, 2016

https://ci.openshift.redhat.com/jenkins/job/fork_ami/276/ is going to be the one, I can feel it!

@deads2k
Copy link
Contributor Author

deads2k commented Aug 10, 2016

upstream is going in. [merge]

@derekwaynecarr
Copy link
Member

LGTM

@openshift-bot
Copy link
Contributor

openshift-bot commented Aug 10, 2016

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

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 3e83a4c

@openshift-bot openshift-bot merged commit 8e8e895 into openshift:master Aug 10, 2016
@deads2k
Copy link
Contributor Author

deads2k commented Aug 10, 2016

@deads2k deads2k deleted the add-cq-metrics 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.

5 participants