From 0835f2e2a831a1dbd4c9169ee5b1e6115ff282da Mon Sep 17 00:00:00 2001 From: Mathieu Mitchell Date: Fri, 12 Apr 2013 11:46:06 -0400 Subject: [PATCH] Handle not having the box and providing it via a box_url. The way this was implemented breaks with a terrible runtime error if you don't have the box already installed nor a box_url in your vagrant file. However, this permits you to provide the openstack api_key and auth_url directly in your box file. This wouldn't be possible if you validated the config prior to handling the box url. The proper way to fix the terrible runtime error would be to check for box_url directly in the HandleBoxUrl method in Vagrant core. Reviewed-by: Philippe Godin Conflicts: lib/vagrant-openstack-plugin/version.rb --- lib/vagrant-openstack-plugin/action.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/vagrant-openstack-plugin/action.rb b/lib/vagrant-openstack-plugin/action.rb index 6656921..6946d4e 100644 --- a/lib/vagrant-openstack-plugin/action.rb +++ b/lib/vagrant-openstack-plugin/action.rb @@ -62,6 +62,7 @@ def self.action_ssh def self.action_up Vagrant::Action::Builder.new.tap do |b| + b.use HandleBoxUrl b.use ConfigValidate b.use Call, IsCreated do |env, b2| if env[:result]