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

vagrant rsync command does not work #137

Closed
bryanwb opened this issue May 6, 2016 · 5 comments
Closed

vagrant rsync command does not work #137

bryanwb opened this issue May 6, 2016 · 5 comments

Comments

@bryanwb
Copy link

bryanwb commented May 6, 2016

This is likely related to #85. The rsync subcommand does not currently work w/ the google provider. The plugin does not generate the .vagrant/machines/default/google/synced_folders file that the vagrant rsync command uses to perform the sync.

@Temikus
Copy link
Collaborator

Temikus commented May 6, 2016

@bryanwb The change has been reverted due to NFS issues: #94

I was hoping there is an official way to fix this, but considering that upstream bug is not getting update a workaround will probably be needed. Something like a custom action "FixSyncedFolders" which would force disable NFS with each folder.

It's on my list of things to fix when I have some cycles.
Meanwhile - patches are very welcome and I can help with code review.

@bryanwb
Copy link
Author

bryanwb commented May 9, 2016

@Temikus tks for the quick reply. I have a workaround atm, but I'll try to come back to this

@numediaweb
Copy link

Any updates on this issue? @Temikus Could you please share your workaround?

@Temikus
Copy link
Collaborator

Temikus commented Feb 2, 2018

@numediaweb Didn't have time to do this yet, I'm afraid. Will be happy to give you pointers or take a look at a PR, if you're willing to do one.

Re:workaround - depends on what you're trying to do. The rsync command is not available but folders do sync via rsync correctly on "up". You need to resync without reprovisioning?

@Temikus Temikus mentioned this issue Mar 23, 2018
@Temikus
Copy link
Collaborator

Temikus commented Mar 28, 2018

@numediaweb @bryanwb This should be fixed, new release that uses built-in rsync is now available.

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

No branches or pull requests

3 participants