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

Feature: Simple health checking #101

Open
birkland opened this issue Jan 18, 2017 · 2 comments
Open

Feature: Simple health checking #101

birkland opened this issue Jan 18, 2017 · 2 comments

Comments

@birkland
Copy link
Contributor

Create a an API-X component that performs basic health checks (e.g periodic HEAD requests) and removes unhealthy services from an LDP service instance registry.

The intent of this is to provide simple-to-use functionality out of the box for those who unable to leverage an existing infrastructure that offers health checking and registry maintenance, such as consul or zookeeper.

It should straightforward to use this or not use it.

This may be demonstrated in a Docker demo this by having the user run a local service (i.e. on their own machine) that self-registers, and is subsequently killed by the user via ctrl-c

So a demo might look like:

  • user starts API-X
  • runs local service on command line which self-registers itself as an extension
  • verify that it's working via API-X
  • Shut down local service
  • Should get 503 error if there are no available services to satisfy the request
@ajs6f
Copy link
Contributor

ajs6f commented Jan 18, 2017

Interesting that that's a 503. If you have time can you point me at the spec that details that?

@birkland
Copy link
Contributor Author

birkland commented Jan 18, 2017

@ajs6f There's nothing in the API-X design doc that spells that out, but this would be a good opportunity to do so. 503 just seemed logical because:

  • There's an extension definition (still) in place (from when the extension self-registered) which spells out the intent that API-X exposes the given service at a particular location, but
  • There are no service instances available to handle the request

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

No branches or pull requests

2 participants