MLServer icon indicating copy to clipboard operation
MLServer copied to clipboard

missing mentioning pushing image to the docker registry in the "serving custom model page"

Open saeid93 opened this issue 3 years ago • 1 comments
trafficstars

Pushing the built docker container to a docker registry is not mentioned after the build process in Serving a custom model. In my case, it was necessary but I wasn't sure if this is a generalizable case to add this as a pull request. If necessary I can add an extra step to the notebook for pushing to the container registry step.

saeid93 avatar Sep 01 '22 20:09 saeid93

Hey @saeid93 ,

That's a great point.

I agree with you though, in that it's not that generalizable. For starters, we would need to know the Docker Hub repo and image name that the user wants to use.

What we could do instead, is to add a "Note block" clarifying that Seldon Core may expect that the image is available on Docker Hub or an alternative Docker registry and therefore the user may need to push it there?

adriangonz avatar Sep 02 '22 08:09 adriangonz