Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Prometheus: add kube-state-metrics #236

Merged
merged 1 commit into from
Nov 23, 2016
Merged

Prometheus: add kube-state-metrics #236

merged 1 commit into from
Nov 23, 2016

Conversation

mgoodness
Copy link
Contributor

As mentioned in #229.

From https://github.com/kubernetes/kube-state-metrics:

kube-state-metrics is a simple service that listens to the Kubernetes API server and generates metrics about the state of the objects.

Also increased truncation to 63 characters.

@k8s-ci-robot
Copy link
Contributor

Can a kubernetes member verify that this patch is reasonable to test? If so, please reply with "@k8s-bot ok to test" on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands will still work. Regular contributors should join the org to skip this step.

If you have questions or suggestions related to this bot's behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Nov 19, 2016
@prydonius
Copy link
Member

@mgoodness can you rebase to resolve the conflict in the README?

As mentioned in #229.

From https://github.com/kubernetes/kube-state-metrics:
> kube-state-metrics is a simple service that listens to the Kubernetes
> API server and generates metrics about the state of the objects.

Also increased truncation to 63 characters.
@mgoodness
Copy link
Contributor Author

You got it. Note that I had a mistake in the README from my other PR. Fixed that while I was in there.

@prydonius
Copy link
Member

@k8s-bot ok to test

@viglesiasce
Copy link
Contributor

Just tested this and seeing the metrics come right up. Thanks @mgoodness!!

@viglesiasce viglesiasce added lgtm Indicates that a PR is ready to be merged. UX reviewed and removed awaiting review labels Nov 22, 2016
@lachie83 lachie83 merged commit fd5613f into helm:master Nov 23, 2016
@mgoodness mgoodness deleted the kube-state-metrics branch November 23, 2016 00:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. code reviewed lgtm Indicates that a PR is ready to be merged. UX reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants