-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Use rsync for remote-run. #41125
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
Use rsync for remote-run. #41125
Conversation
@swift-ci Please test |
@swift-ci Please smoke test |
9c1c531
to
738cebf
Compare
@swift-ci Please test |
@swift-ci Please smoke 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.
I love it.
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.
Kinda sad that this makes remote testing more challenging for Windows, but given that isn't in use right now ... we can deal with that later.
Build failed |
Not only does this give us a huge speed-up, it also works around a problem where sftp doesn't always update the modification time, which causes random test failures. rdar://88179140
738cebf
to
2f6f051
Compare
:-D Would have helped if I'd checked in the @swift-ci Please smoke test |
@swift-ci Please test |
There are Windows versions of |
@swift-ci please test Windows platform |
Build failed |
@swift-ci Please test macOS platform |
Not only does this give us a huge speed-up, it also works around a problem where
sftp
doesn't always update the modification time, which causes random test failures.rdar://88179140