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

Use fog-brightbox gem #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tokengeek
Copy link

This changes the dependencies to rely on the fog-brightbox gem which is a module subset of the full fog gem. This has lighter dependencies and requires less code on starting.

Also replaces the now deprecated HandleBoxUrl action with HandleBox following warnings from vagrant 1.5.

Whitespace is also cleaned up.

Rather than requiring the entire `fog` gem this uses the Brightbox
module to cut down on dependencies.
@tokengeek
Copy link
Author

There might be an issue related to provisioning with vagrant-v0.15.0.dev or earlier.

If I include a provisioning step, vagrant up bombs out.

There was an error talking to the Brightbox cloud. The error
message is shown below:

wrong number of arguments (1 for 3)

Working past the error handling and https://github.com/NeilW/vagrant-brightbox/blob/master/lib/vagrant-brightbox/action/timed_provision.rb#L9 seems to be getting 1 argument now.

This is actually happening on master when testing this as a gem or bundled.

So if you see this it might be an upstream issue. I've just not tracked it down enough to report.

@tokengeek
Copy link
Author

I believe the issue is upstream - hashicorp/vagrant#3226

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.

1 participant