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

flanneld: adds basic health-check endpoint #632

Closed

Conversation

genevieve
Copy link

This begins to address #616.

We can follow up later with a richer health-check that captures connectivity to etcd.

cc: @rosenhouse

Begins to address flannel-io#616

Signed-off-by: Gabe Rosenhouse <[email protected]>
@andyxning
Copy link
Contributor

Any updates on this?

@genevieve
Copy link
Author

@rosenhouse ^^^

@genevieve genevieve closed this Mar 24, 2017
@andyxning
Copy link
Contributor

@genevievelesperance Why you close this PR? Is healthcheck functionality available already?

@rosenhouse
Copy link

@andyxning Our team has recently decided to stop using flannel and to build something different. So we won't be working on a flannel healthcheck.

Feel free to open a new PR with this code or something similar. We just won't be shepherding it.

@andyxning andyxning mentioned this pull request May 17, 2017
@rosenhouse rosenhouse deleted the health-check branch June 6, 2017 19:34
@rosenhouse rosenhouse restored the health-check branch June 22, 2017 00:16
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