Skip to content
This repository has been archived by the owner on Dec 1, 2018. It is now read-only.

Add "/healthz" endpoint to Heapster. #1271

Merged
merged 2 commits into from
Aug 31, 2016

Conversation

mksalawa
Copy link
Contributor

@mksalawa mksalawa commented Aug 29, 2016

Create a "/healthz" endpoint and a handler checking whether the latest data batch was gathered within a given period and whether there are any metrics available.

This change is Reviewable

@k8s-bot
Copy link

k8s-bot commented Aug 29, 2016

Jenkins GCE e2e

Build/test failed for commit 522c811.

@mksalawa
Copy link
Contributor Author

cc @piosz @mwielgus

@mksalawa mksalawa changed the title Add liveness probe. Add "/healthz" endpoint to Heapster. Aug 29, 2016
@k8s-bot
Copy link

k8s-bot commented Aug 30, 2016

Jenkins GCE e2e

Build/test failed for commit 162fe81.

@mksalawa mksalawa closed this Aug 30, 2016
@mksalawa mksalawa reopened this Aug 30, 2016
@mksalawa
Copy link
Contributor Author

@k8s-bot test this please

@k8s-bot
Copy link

k8s-bot commented Aug 30, 2016

Jenkins GCE e2e

Build/test failed for commit 162fe81.

@k8s-bot
Copy link

k8s-bot commented Aug 30, 2016

Jenkins GCE e2e

Build/test failed for commit 48925dc.

@k8s-bot
Copy link

k8s-bot commented Aug 30, 2016

Jenkins GCE e2e

Build/test failed for commit bd1db1e.

@k8s-bot
Copy link

k8s-bot commented Aug 30, 2016

Jenkins GCE e2e

Build/test failed for commit 77c6e23.

@k8s-bot
Copy link

k8s-bot commented Aug 30, 2016

Jenkins GCE e2e

Build/test passed for commit ae8b6ca.

@k8s-bot
Copy link

k8s-bot commented Aug 31, 2016

Jenkins GCE e2e

Build/test passed for commit e61cc25.

@piosz
Copy link
Contributor

piosz commented Aug 31, 2016

LGTM, thanks!

@piosz piosz added the lgtm Indicates that a PR is ready to be merged. label Aug 31, 2016
@piosz piosz merged commit e9d71bb into kubernetes-retired:master Aug 31, 2016
@AlmogBaku
Copy link
Contributor

@piosz @mksalawa what about eventer? it's sounds to me like a similar solution for eventer could be great

@piosz
Copy link
Contributor

piosz commented Sep 8, 2016

Please create an issue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants