-
Notifications
You must be signed in to change notification settings - Fork 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
[ci] Fix several release tests #37022
Conversation
071220b
to
2d4130a
Compare
@@ -3676,7 +3676,7 @@ | |||
script: pytest checkpointing_tests/test_learner_group_checkpointing.py | |||
|
|||
wait_for_nodes: | |||
num_nodes: 3 | |||
num_nodes: 2 |
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.
why is this node number change required?
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.
A previous PR reduces the number of node to 2 #36929, so this needs to update as well
https://buildkite.com/ray-project/release-tests-pr/builds/44305 is still failing? |
21bee42
to
3b46add
Compare
Signed-off-by: can <can@anyscale.com>
3b46add
to
f054cd7
Compare
Some tests still fail but they are the second layer of failures after my fix. I'll leave them for rllib folks to debug them further then. |
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.
Signed-off-by: can <can@anyscale.com> Signed-off-by: e428265 <arvind.chandramouli@lmco.com>
Fix several release tests. I need to send out instructions on how to write tests with byod soon so people avoid breaking it.
Testing Strategy