search

mitchellh / vagrant-rackspace

Use Vagrant to manage Rackspace Cloud instances.
MIT License
236 stars 155 forks source link

Upgrade minimum Vagrant version from 1.1 to 1.5 #129

Closed maxlinc closed 9 years ago

maxlinc commented 9 years ago

@smashwilson / @mdarby WIP but ready for review. I want to get your review and feedback on the changes so far, but check w/ me before merging so I can do one last review and cleanup.

Fixes #125.

maxlinc commented 9 years ago

This may fix:

Those issues were related to retry logic for communicators (winrm or ssh).

This PR replaces legacy code to wait for a communicator with the built-in WaitForCommunicator action. I think the built-in action may be smarter about when to fail fast and when to keep retrying.

smashwilson commented 9 years ago

@maxlinc :shipit: