Skip to content

[autoscaler] Enable Rsync with Docker Containers #8527

Closed
ijrsvt wants to merge 4 commits intoray-project:masterfrom
ijrsvt:rsync_docker
Closed

[autoscaler] Enable Rsync with Docker Containers #8527
ijrsvt wants to merge 4 commits intoray-project:masterfrom
ijrsvt:rsync_docker

Conversation

@ijrsvt
Copy link
Copy Markdown
Contributor

@ijrsvt ijrsvt commented May 20, 2020

Why are these changes needed?

Ray cannot currently rsync into or out of docker containers. This is a problem also for ray submit which uses rsync;

Related issue number

Closes #4183

Checks

  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/latest/.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failure rates at https://ray-travis-tracker.herokuapp.com/.
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested (please justify below)

@AmplabJenkins
Copy link
Copy Markdown

Can one of the admins verify this patch?

@ijrsvt ijrsvt requested review from ericl, richardliaw and simon-mo and removed request for ericl May 20, 2020 23:03
@AmplabJenkins
Copy link
Copy Markdown

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

@AmplabJenkins
Copy link
Copy Markdown

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

@AmplabJenkins
Copy link
Copy Markdown

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

@simon-mo
Copy link
Copy Markdown
Contributor

simon-mo commented Jun 1, 2020

Do you want to open a new PR to implement a CommandRunner and close this one? it's OK to re-use this one as well.

@ijrsvt
Copy link
Copy Markdown
Contributor Author

ijrsvt commented Jun 1, 2020 via email

@ijrsvt ijrsvt closed this Jun 1, 2020
@ijrsvt ijrsvt deleted the rsync_docker branch August 26, 2020 00:24
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.

[autoscaler] Ray submit with docker doesn't rsync the script to the container

3 participants