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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

docs: fix bentoml containerize command help message #2957

Merged
merged 5 commits into from Aug 31, 2022
Merged

docs: fix bentoml containerize command help message #2957

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
e63b61c
docs: update docstring
aarnphm Aug 30, 2022
c393c5b
fix(logs): wrong formatted message
aarnphm Aug 30, 2022
8dc839d
fix: backticks to quotes
aarnphm Aug 30, 2022
1b0cc02
chore: Update bentoml/bentos.py
aarnphm Aug 31, 2022
6e84997
chore: update tag to be list type
aarnphm Aug 31, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
10 changes: 8 additions & 2 deletions bentoml/bentos.py
View file
Open in desktop
Expand Up @@ -465,9 +465,15 @@ def containerize(
)
return False
else:
logger.info(f'Successfully built docker image "{docker_image_tag}"')
custom_tags = False if not isinstance(docker_image_tag, str) else True
aarnphm marked this conversation as resolved.
Show resolved Hide resolved
logger.info(
f'To run your newly built Bento container, use: "docker run -it --rm -p 3000:3000 {docker_image_tag}"'
'Successfully built docker image for "%s" with tags "%s"',
str(bento.tag),
docker_image_tag if not custom_tags else ",".join(docker_image_tag),
Copy link
Contributor

Choose a reason for hiding this comment

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

It seems to me like we should make this variable always an array.

Copy link
Member Author

Choose a reason for hiding this comment

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

I remember we have this discussion before, let me do that.

)
logger.info(
'To run your newly built Bento container, use: "docker run -it --rm -p 3000:3000 %s"',
docker_image_tag[0] if custom_tags else docker_image_tag,
)
return True

Expand Down
4 changes: 2 additions & 2 deletions bentoml_cli/containerize.py
View file
Open in desktop
Expand Up @@ -196,11 +196,11 @@ def containerize( # type: ignore
BENTO is the target BentoService to be containerized, referenced by its name
and version in format of name:version. For example: "iris_classifier:v1.2.0"

`bentoml containerize` command also supports the use of the `latest` tag
'bentoml containerize' command also supports the use of the 'latest' tag
which will automatically use the last built version of your Bento.

You can provide a tag for the image built by Bento using the
`--tag` flag. Additionally, you can provide a `--push` flag,
`--docker-image-tag` flag. Additionally, you can provide a `--push` flag,
aarnphm marked this conversation as resolved.
Show resolved Hide resolved
which will push the built image to the Docker repository specified by the
image tag.

Expand Down