Fix folder syncronization with ipv6-only hosts

ggiamarchi / vagrant-openstack-provider

Use Vagrant to manage OpenStack Cloud instances.

MIT License

245 stars 102 forks source link

Fix folder syncronization with ipv6-only hosts #305

Closed mechmind closed 7 years ago

mechmind commented 7 years ago

Hi Guillaume. We are about to adopt your plugin but discovered that in our ipv6-only cloud it will not handle syncronization properly. So there are a fix that encloses ipv6 address in braces and make ssh happy.

coveralls commented 7 years ago

Coverage decreased (-0.04%) to 91.592% when pulling 1a9925662c73fd8d0eb429e853591ca3dab63c0a on mechmind:rsync-to-ipv6-only-host into 4a252918e7ebbed0db5562dd8f671bccadde586e on ggiamarchi:master.

ggiamarchi commented 7 years ago

@mechmind The provider has moved now to the standard vagrant synced folders middleware. The legacy rsync middleware within this provider is still available for now (use_legacy_synced_folders = true) but will be removed in a further release. By the way, the legacy implementation is no longer maintained. Thanks for your contribution.

mechmind commented 7 years ago

OK. I see current version of plugin is still 0.7.2. Do you plan to release updated version soon?

ggiamarchi commented 7 years ago

Yes, a version will be released by the end of the week

mechmind commented 7 years ago

Thanks!