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 check to Dockerfile #2219

Open
alejones opened this issue Apr 18, 2024 · 4 comments
Open

Add health check to Dockerfile #2219

alejones opened this issue Apr 18, 2024 · 4 comments

Comments

@alejones
Copy link

Feature Request

Add a health check to the Dockerfile

Describe the problem the feature is intended to solve

I'm using a docker container on an edge device and would like to be able to shut it down if it becomes unresponsive.

Describe the solution

Inside the docker file add a HEALTHCHECK section. I'll leave it up to the TensorFlow team to decide how it should be checked. Just checking the container is alive would be a start, actually checking that it can return predictions would be even better.

For checking if it is alive, I use this in Jupyter notebooks. This isn't a complete solution, but just an idea.

curl http://10.0.0.10:8501/v1/models/my_model

For checking if predictions can be returned, I would be happy with a prediction from the half plus two model that is used when testing the dockerfile.

This is an exert from the linked Docker documentation on how to add health checks.

HEALTHCHECK --interval=5m --timeout=3s \
  [CMD](https://docs.docker.com/reference/dockerfile/#cmd) curl -f http://localhost/ || exit 1

Describe alternatives you've considered

None yet. Open to suggestions on how to restart a contain started with systemd

Additional context

I'm using rootless Podman and Ubuntu 24.04 on an edge device. I'd like to use the health checking built into Podman to be able to kill the container, and then have systemd bring up a fresh container. I have no problems starting and using TensorFlow Serving with Podman, but monitoring does not currently work for me.

@alejones alejones changed the title Healthcheck to Dockerfile Add health check to Dockerfile Apr 18, 2024
@alejones
Copy link
Author

Any updates on this?

@alejones
Copy link
Author

Checking again if there any any udates.

@YanghuaHuang
Copy link
Member

Sorry about the late reply. I personally didn't get the chance and bandwidth to look into this. Unassign myself as I am not actively working on this.

@YanghuaHuang YanghuaHuang removed their assignment May 23, 2024
@alejones
Copy link
Author

Thanks for the update, just want to make sure it doesn't get lost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants