-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
[release byod] fix ml requirements file selection #39353
Conversation
one more place that I found about picking ml-or-not that is based on the old "cpu-or-not" logic. |
c4a2bce
to
e42da28
Compare
@@ -251,15 +251,15 @@ def get_byod_image_tag(self) -> str: | |||
} | |||
return f"{self.get_byod_base_image_tag()}-{dict_hash(custom_info)}" | |||
|
|||
def _use_byod_ml_image(self) -> bool: | |||
def use_byod_ml_image(self) -> bool: |
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.
maybe if using ml image should have it's own field? like technically speaking, we also have ray-ml:cpu
image, just never used in release tests..
this probably only affects @xwjiang2010 's new test. |
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.
release/ray_release/byod/build.py
Outdated
if test.get_byod_type() == "cpu" | ||
else f"{REQUIREMENTS_ML_BYOD}_{test.get('python', PYTHON_VERSION)}.txt" | ||
) | ||
py_version = test.get("python", PYTHON_VERSION) |
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.
nit: test.get_python_version()
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.
done.. why were you not using that in the past?
and also change ecr to registry, as it might be google's too. Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
e42da28
to
fbcb2c2
Compare
and also change ecr to registry, as it might be google's too. Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
and also change ecr to registry, as it might be google's too. Signed-off-by: Lonnie Liu <lonnie@anyscale.com> Signed-off-by: Jim Thompson <jimthompson5802@gmail.com>
and also change ecr to registry, as it might be google's too. Signed-off-by: Lonnie Liu <lonnie@anyscale.com> Signed-off-by: Victor <vctr.y.m@example.com>
and also change ecr to registry, as it might be google's too.