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

Fix local rsync issues #1372

Merged
merged 4 commits into from
Jul 2, 2020
Merged

Fix local rsync issues #1372

merged 4 commits into from
Jul 2, 2020

Conversation

ferki
Copy link
Member

@ferki ferki commented Jul 2, 2020

This PR fixes #1370 by testing for, and fixing the following issues around local rsync operations:

  • if {download => 1} is in effect, don't prefix source with user@servername:
  • servername for local connections inside tasks is <local>, so check for that too

I open this PR as draft as it most probably needs some clean up (changelog, wording, also more general testing approach for the various options) before merging.

Checklist

  • based on top of latest source code
  • changelog entry included
  • tests pass on Travis CI
  • git history is clean
  • git commit messages are well-written

@sdondley
Copy link
Contributor

sdondley commented Jul 2, 2020

Tested manually and it works. I also ran dzil test locally and all tests passed.

Thanks!

@ferki
Copy link
Member Author

ferki commented Jul 2, 2020

Tested manually and it works. I also ran dzil test locally and all tests passed.

Great, thanks for the feedback! I'll try to include it in a last trial release before 1.12.0 later today.

@ferki ferki force-pushed the local_rsync_fixes branch from c9b5eda to cc1cac6 Compare July 2, 2020 14:48
@sdondley
Copy link
Contributor

sdondley commented Jul 2, 2020

Any chance you can look at #1356?

@ferki
Copy link
Member Author

ferki commented Jul 2, 2020

Any chance you can look at #1356?

That's a different story and context. At the moment, I don't expect that I'll be able to handle that before the next release.

@ferki ferki marked this pull request as ready for review July 2, 2020 16:25
@ferki ferki merged commit 0c47af0 into master Jul 2, 2020
@ferki ferki deleted the local_rsync_fixes branch July 2, 2020 16:39
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.

running sync() locally results in errors
2 participants