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.

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

Add a new autoscaling shuffle test #16948

Merged
merged 7 commits into from
Jul 12, 2021
Merged

Conversation

ericl
Copy link
Contributor

@ericl ericl commented Jul 8, 2021

Why are these changes needed?

This adds an autoscaling shuffler test.

@ericl ericl added the tests-ok The tagger certifies test failures are unrelated and assumes personal liability. label Jul 8, 2021
head_node_type:
name: head_node
instance_type: i3.4xlarge
resources: {"object_store_memory": 21474836480}
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess this is copied from other config, but we can remove this now since SIGBUS is properly handled

@ericl
Copy link
Contributor Author

ericl commented Jul 11, 2021

@rkooo567 can you add this to the releaser/ repo before merge? (remind me why that repo isn't part of ray/ again?)

@rkooo567
Copy link
Contributor

on it now. Give me 10 minutes.

@rkooo567
Copy link
Contributor

For why it is separated, I am not 100% sure. In the past it was separated because it was an experimental project

@rkooo567
Copy link
Contributor

Done

@ericl
Copy link
Contributor Author

ericl commented Jul 12, 2021

Merge conflict

@ericl ericl removed the tests-ok The tagger certifies test failures are unrelated and assumes personal liability. label Jul 12, 2021
@ericl ericl merged commit fa0ff05 into ray-project:master Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants