The Wayback Machine - https://web.archive.org/web/20201125121412/https://github.com/smartcontractkit/chainlink/issues/3631
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

[FEAT] Implement Docker healthcheck #3631

Open
willianpaixao opened this issue Nov 13, 2020 · 2 comments · May be fixed by #3632
Open

[FEAT] Implement Docker healthcheck #3631

willianpaixao opened this issue Nov 13, 2020 · 2 comments · May be fixed by #3632

Comments

@willianpaixao
Copy link
Contributor

@willianpaixao willianpaixao commented Nov 13, 2020

Description
Add a simple /healthcheck endpoint that simply returns 200 in case no error has been reported. In case of error, a different return code and a JSON returns otherwise.

Motivation
To easily spot and automatically restart unresponsive containers.

Justification
Such a feature would improve the monitoring of the chainlink node, letting your monitoring tool quick know and restart a given container. [1]
Most modern orchestration tools (ECS, K8S, etc) can read this health status and automatically restart the service. [2] [3]

Additional Information
If the application is still responsive (working normally), a docker ps would indicate the healthy status:

CONTAINER ID        IMAGE                            COMMAND                  CREATED             STATUS                   PORTS               NAMES
e2722770cc7f        smartcontract/chainlink:latest   "chainlink local node"   3 minutes ago       Up 3 minutes (healthy)                       chainlink
willianpaixao added a commit to willianpaixao/chainlink that referenced this issue Nov 13, 2020
Fix smartcontractkit#3631

Implement a /healthcheck endpoint and adds a Docker healthcheck check.
@willianpaixao willianpaixao linked a pull request that will close this issue Nov 13, 2020
@hershkoy
Copy link

@hershkoy hershkoy commented Nov 15, 2020

I want this too

@PatrickAlphaC
Copy link

@PatrickAlphaC PatrickAlphaC commented Nov 19, 2020

Thanks for this issue @willianpaixao! Added it as an enhancement and good first issue

willianpaixao added a commit to willianpaixao/chainlink that referenced this issue Nov 24, 2020
Fix smartcontractkit#3631

Implement a /healthcheck endpoint and adds a Docker healthcheck check.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.