search

bentoml / bentoctl

Fast model deployment on any cloud 🚀
https://bentoml.com
Other
175 stars 30 forks source link

Error [BentoctlDockerException]: Failed to tag Docker image, google-compute-engine-iris_classifier:6rqjqqbhbsscvnjh not found. #189

Closed math-sasso closed 1 year ago

math-sasso commented 1 year ago

*Describe the bug When I call bentoctl build as the command below to push an image to my container registry I am always getting this same error. 

bentoctl build -b iris_classifier:6rqjqqbhbsscvnjh -f deployment_config.yaml
Created the repository outside-venv-test
Error [BentoctlDockerException]:  Failed to tag Docker image, google-compute-engine-iris_classifier:6rqjqqbhbsscvnjh not found.

To Reproduce

What is curios is that my teammates can run it in their computers. My machine is a Ubuntu 20.04. I installed docker, gcloud and terraform according to those tools documentation

Follow the bentoml tutorial runnning those steps.

  1. bentoml build
  2. git clone https://github.com/bentoml/google-compute-engine-deploy.git
  3. bentoctl operator install google-compute-engine-deploy
  4. bentoctl build -b iris_classifier:6rqjqqbhbsscvnjh -f deployment_config.yaml
  5. see error

image

Environment:

math-sasso commented 1 year ago

My work around is to first call docker containerize to create a docker image and after tag the image in the format expected by bentoctl. This should be done automatically by bentoctl, I still dont know the reason it fails on my machine. 

bentoml containerize iris_classifier:6rqjqqbhbsscvnjh
docker tag iris_classifier:6rqjqqbhbsscvnjh google-compute-engine-iris_classifier:6rqjqqbhbsscvnjh
dbuades commented 1 year ago

I found the same problem today in Mac using https://github.com/bentoml/aws-ec2-deploy and fixed it by using the same workaround.

jjmachan commented 1 year ago

This issue should now be fixed in the main branch. @math-sasso @dbuades can you try it out and verify if that works for you? The installation instructions are mentioned here https://github.com/bentoml/bentoctl/blob/main/DEVELOPMENT.md#install-bentoctl-from-source

and thank you for reporting this issue. if this fixes your issue we will cut a new release for this fix.

jjmachan commented 1 year ago

PR #187 should have fixed this

dbuades commented 1 year ago

@jjmachan I confirm this fixes the issue, thanks! Sorry for the delay

jjmachan commented 1 year ago

thanks for confirming @dbuades. Closing this issue now