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 health endpoint #151

Merged
merged 2 commits into from
Nov 18, 2019
Merged

Add health endpoint #151

merged 2 commits into from
Nov 18, 2019

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Nov 14, 2019

Adding the /health endpoint which can be used for Docker / K8s deployment. There are two common calls:

  • /health for liveliness
  • /health?ready=1 for a ready check

Currently both are implemented the same way as as soon as the service is live, it is also ready. This might change in the future.

Adding the `/health` endpoint which can be used for Docker / K8s deployment. There are two common calls:

* `/health` for liveliness
* `/health?ready=true` for a ready check

Currently both are implemented the same way as as soon as the service is live, it is also ready. This might change in the future.
@ruflin ruflin requested a review from JoshMock November 14, 2019 08:03
Copy link
Member

@JoshMock JoshMock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks perfect. Thanks @ruflin!

@ruflin ruflin merged commit 4ede7e6 into elastic:master Nov 18, 2019
@ruflin ruflin deleted the health-endpoint branch November 18, 2019 08:46
cleydyr added a commit to elastic/observability-docs that referenced this pull request Dec 16, 2022
cleydyr added a commit to elastic/observability-docs that referenced this pull request Jan 4, 2023
mergify bot pushed a commit to elastic/observability-docs that referenced this pull request Jan 4, 2023
mergify bot pushed a commit to elastic/observability-docs that referenced this pull request Jan 4, 2023
mergify bot pushed a commit to elastic/observability-docs that referenced this pull request Jan 4, 2023
cleydyr added a commit to elastic/observability-docs that referenced this pull request Jan 4, 2023
…2468)

Since elastic/package-registry#151

(cherry picked from commit e475f24)

Co-authored-by: Cleydyr Bezerra de Albuquerque <cleydyr.bezerradealbuquerque@elastic.co>
cleydyr added a commit to elastic/observability-docs that referenced this pull request Jan 4, 2023
…2469)

Since elastic/package-registry#151

(cherry picked from commit e475f24)

Co-authored-by: Cleydyr Bezerra de Albuquerque <cleydyr.bezerradealbuquerque@elastic.co>
dedemorton pushed a commit to elastic/observability-docs that referenced this pull request Jan 4, 2023
…2470)

Since elastic/package-registry#151

(cherry picked from commit e475f24)

Co-authored-by: Cleydyr Bezerra de Albuquerque <cleydyr.bezerradealbuquerque@elastic.co>
bmorelli25 pushed a commit to bmorelli25/observability-docs that referenced this pull request Apr 11, 2023
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.

2 participants