-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Adding image_uri to docstring of runtime_env #47905
base: master
Are you sure you want to change the base?
Conversation
Adding image_uri description and example to docstring Signed-off-by: Japneet <[email protected]>
Co-authored-by: angelinalg <[email protected]> Signed-off-by: Japneet <[email protected]>
Co-authored-by: angelinalg <[email protected]> Signed-off-by: Japneet <[email protected]>
@@ -263,6 +267,9 @@ class MyClass: | |||
config: config for runtime environment. Either | |||
a dict or a RuntimeEnvConfig. Field: (1) setup_timeout_seconds, the | |||
timeout of runtime environment creation, timeout is in seconds. | |||
image_uri: URI to a container image. The Ray worker process runs | |||
in a container with this image. You can't use this parameter with other fields | |||
of runtime_env. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you specify what happens when the image cannot be found? does it raise an exception?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RuntimeError: {"message":"failed to pull image after 5 retries: containerd client pull: failed to resolve reference "docker.io/<image_uri>": docker.io/<image_uri>: not found"}
Reduce line length Signed-off-by: Japneet <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shortened line length
Adding
image_uri
description and example to docstring. The parameter is in the signature, but not in the description section for the parameters.cc: @jjyao
Checks
git commit -s
) in this PR.scripts/format.sh
to lint the changes in this PR.method in Tune, I've added it in
doc/source/tune/api/
under thecorresponding
.rst
file.