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

Add /live health endpoint to admin server #221

Merged
merged 1 commit into from
Mar 30, 2021
Merged

Conversation

markmandel
Copy link
Contributor

Create a Health module which tracks if a panic occurs anywhere in the code base (which may or may not be on the main thread),
and moves the system to unhealthy.

In the future we could add extra checks to this module as we discover more things that impact proxy health.

Closes #73

Create a `Health` module which tracks if a panic occurs
anywhere in the code base (which may or may not be on the main thread),
and moves the system to unhealthy.

In the future we could add extra checks to this module as we discover
more things that impact proxy health.

Closes #73
@markmandel markmandel added kind/feature New feature or request area/operations Installation, updating, metrics etc labels Mar 30, 2021
@markmandel markmandel requested a review from iffyio March 30, 2021 18:22
@google-cla google-cla bot added the cla: yes label Mar 30, 2021
@markmandel markmandel merged commit 1ece2db into main Mar 30, 2021
@markmandel markmandel deleted the feature/live-endpoint branch March 30, 2021 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/operations Installation, updating, metrics etc cla: yes kind/feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add health endpoint
2 participants