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

Provide an all in one prometheus template example #12793

Merged
merged 1 commit into from
Feb 5, 2017

Conversation

smarterclayton
Copy link
Contributor

@smarterclayton smarterclayton commented Feb 3, 2017

Requires cluster-reader to be added to the prometheus endpoint

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 21eb74b

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

description: |
A monitoring solution for an OpenShift cluster - collect and gather metrics from nodes, services, and the infrastructure.
You will need to grant access to the prometheus service account to access nodes, metrics endpoints, and services across
namespaces - use `oadm policy add-cluster-role-to-user system:serviceaccount:NAMESPACE:prometheus cluster-reader`.
Copy link

Choose a reason for hiding this comment

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

should be oadm policy add-cluster-role-to-user cluster-reader system:serviceaccount:NAMESPACE:prometheus

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge Running (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/13559/) (Base Commit: e2e5d9e)

@smarterclayton
Copy link
Contributor Author

Yes it should, thanksz

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 3052bd6

@openshift-bot
Copy link
Contributor

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

@smarterclayton smarterclayton merged commit 6f30f7e into openshift:master Feb 5, 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