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

Add missing permission when running 'oc cluster up --metrics' #11860

Merged
merged 1 commit into from
Nov 10, 2016

Conversation

mwringe
Copy link
Contributor

@mwringe mwringe commented Nov 9, 2016

No description provided.

@csrwng
Copy link
Contributor

csrwng commented Nov 9, 2016

@mwringe looks like gofmt is complaining, but other than that, LGTM

@mwringe mwringe force-pushed the cluster-up-hawkular-sa branch from f32595b to 127e302 Compare November 9, 2016 22:27
@mwringe
Copy link
Contributor Author

mwringe commented Nov 10, 2016

@csrwng ok, fixed. Is there anything else that I can do to get this merged?

@csrwng
Copy link
Contributor

csrwng commented Nov 10, 2016

@bparees ptal

@bparees
Copy link
Contributor

bparees commented Nov 10, 2016

[merge]

@bparees bparees self-assigned this Nov 10, 2016
@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@bparees
Copy link
Contributor

bparees commented Nov 10, 2016

flake #11873
flake #9076

[merge]
[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 127e302

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 127e302

@openshift-bot
Copy link
Contributor

openshift-bot commented Nov 10, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11378/) (Base Commit: 7564399) (Image: devenv-rhel7_5349)

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11377/) (Base Commit: 7564399)

@openshift-bot openshift-bot merged commit c2ffe41 into openshift:master Nov 10, 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.

4 participants