Add support for availability zone

This is needed to choose which compute node you will boot on.
Not added test to create_server, this is tech debt
we should address soon.

Uses the fog default if the availability zone is not passed, should
not break anything.

Tested locally with and without the availability_zone config.

Author: fbouliane

README.md

@@ -70,6 +70,8 @@ This provider exposes quite a few provider-specific configuration options:
 * `image` - The server image to boot. This can be a string matching the
   exact ID or name of the image, or this can be a regular expression to
   partially match some image.
+* `availability_zone` - The availability zone to use with nova, 
+this allows to choose which zone your instance will exist on.
 * `endpoint` - The keystone authentication URL of your OpenStack installation.
 * `server_name` - The name of the server within the OpenStack Cloud. This
   defaults to the name of the Vagrant machine (via `config.vm.define`), but

lib/vagrant-openstack-cloud-provider/action/create_server.rb

@@ -64,6 +64,11 @@ def call(env)
               env[:ui].info("      - #{net['name']}")
             end
           end
+
+          unless config.availability_zone.nil?
+            env[:ui].info(" -- Availability Zone: #{config.availability_zone}")
+          end
+
           env[:ui].info(" -- Name: #{server_name}")
 
           openstack_nics = []
@@ -83,6 +88,10 @@ def call(env)
             :os_scheduler_hints => config.scheduler_hints
           }
 
+          unless config.availability_zone.nil?
+            options[:availability_zone] = config.availability_zone
+          end
+
           if openstack_nics.any?
             options[:nics] = openstack_nics
           end

lib/vagrant-openstack-cloud-provider/config.rb

@@ -94,6 +94,9 @@ class Config < Vagrant.plugin("2", :config)
       # @return [String]
       attr_accessor :user_domain_id
 
+      # @return [String]
+      attr_accessor :availability_zone
+
       # Version to use for keystone authentication
       # Not used for now, will be supported in the next version.
       # Known version are 'v2.0', 'v3'
@@ -141,6 +144,7 @@ def initialize
         @instance_ssh_timeout = UNSET_VALUE
         @instance_ssh_check_interval = UNSET_VALUE
         @report_progress = UNSET_VALUE
+        @availability_zone = UNSET_VALUE
 
         @project_name = UNSET_VALUE
         @project_id = UNSET_VALUE
@@ -194,6 +198,7 @@ def finalize!
         @instance_ssh_timeout = 120 if @instance_ssh_timeout == UNSET_VALUE
         @instance_ssh_check_interval = 2 if @instance_ssh_check_interval == UNSET_VALUE
 
+        @availability_zone = nil if @availability_zone == UNSET_VALUE
         @report_progress = true if @report_progress == UNSET_VALUE
       end
 

spec/vagrant-openstack-cloud-provider/config_spec.rb

@@ -40,6 +40,7 @@
     it { is_expected.to have_attributes(identity_api_version: nil) }
 
     it { is_expected.to have_attributes(scheduler_hints: {}) }
+    it { is_expected.to have_attributes(availability_zone: nil) }
     it { is_expected.to have_attributes(instance_build_timeout: 120) }
     it { is_expected.to have_attributes(instance_build_status_check_interval: 1) }
     it { is_expected.to have_attributes(instance_ssh_timeout: 120) }
@@ -71,6 +72,7 @@
       :user_domain_id,
       :identity_api_version,
       :scheduler_hints,
+      :availability_zone,
       :report_progress].each do |attribute|
       it "should not default #{attribute} if overridden" do
         subject.send("#{attribute}=", "foo")