Include interface driver for vpn agent

Fix to include interface driver in the vpn_agent.ini config file. Defaults to ovs interface driver. Change-Id: I1b57817b79627878eac6fcb4cedaad835bdf98c1 Closes-Bug: #1301418
xbezdick · Apr 4, 2014 · cb95f94 · cb95f94
1 parent 55f952d
commit cb95f94
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/manifests/agents/vpnaas.pp b/manifests/agents/vpnaas.pp
@@ -30,13 +30,17 @@
 # [*vpn_device_driver*]
 #   (optional) Defaults to 'neutron.services.vpn.device_drivers.ipsec.OpenSwanDriver'.
 #
+# [*interface_driver*]
+#  (optional) Defaults to 'neutron.agent.linux.interface.OVSInterfaceDriver'.
+#
 # [*ipsec_status_check_interval*]
 #   (optional) Status check interval. Defaults to '60'.
 #
 class neutron::agents::vpnaas (
   $package_ensure              = present,
   $enabled                     = true,
   $vpn_device_driver           = 'neutron.services.vpn.device_drivers.ipsec.OpenSwanDriver',
+  $interface_driver            = 'neutron.agent.linux.interface.OVSInterfaceDriver',
   $ipsec_status_check_interval = '60'
 ) {
 
@@ -64,6 +68,7 @@
   neutron_vpnaas_agent_config {
     'vpnagent/vpn_device_driver':        value => $vpn_device_driver;
     'ipsec/ipsec_status_check_interval': value => $ipsec_status_check_interval;
+    'DEFAULT/interface_driver':          value => $interface_driver;
   }
 
   if $::neutron::params::vpnaas_agent_package {

diff --git a/spec/classes/neutron_agents_vpnaas_spec.rb b/spec/classes/neutron_agents_vpnaas_spec.rb
@@ -34,6 +34,7 @@
     { :package_ensure              => 'present',
       :enabled                     => true,
       :vpn_device_driver           => 'neutron.services.vpn.device_drivers.ipsec.OpenSwanDriver',
+      :interface_driver            => 'neutron.agent.linux.interface.OVSInterfaceDriver',
       :ipsec_status_check_interval => '60'
     }
   end
@@ -51,6 +52,7 @@
     it 'configures vpnaas_agent.ini' do
       should contain_neutron_vpnaas_agent_config('vpnagent/vpn_device_driver').with_value(p[:vpn_device_driver]);
       should contain_neutron_vpnaas_agent_config('ipsec/ipsec_status_check_interval').with_value(p[:ipsec_status_check_interval]);
+      should contain_neutron_vpnaas_agent_config('DEFAULT/interface_driver').with_value(p[:interface_driver]);
     end
 
     it 'installs neutron vpnaas agent package' do