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

Don't delete files in rsync up, and also shorten timeout #4688

Merged
merged 2 commits into from
Apr 25, 2019

Conversation

ericl
Copy link
Contributor

@ericl ericl commented Apr 23, 2019

cc @hartikainen @ls-daniel

10 s should be sufficient to speed up connections, yet hopefully never cause hangs.

Also, remove the delete flag from file mounts; this is dangerous because it can delete files created by the user on the remote end.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-Perf-Integration-PRB/573/
Test PASSed.

@ericl ericl changed the title Even shorter ControlPersist timeout to avoid SSH hangs Don't delete files in rsync up, and also shorten timeout Apr 23, 2019
@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-Perf-Integration-PRB/574/
Test PASSed.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/13939/
Test FAILed.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/13940/
Test FAILed.

@ericl ericl merged commit 47cca97 into ray-project:master Apr 25, 2019
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