Add ability to manage Neutron ML2 plugin package state

Introduce the package_ensure parameter to add the ability
to manage Neutron ML2 plugin package state.

Closes-bug: #1355370
Change-Id: I3bf6ed68015c13cf61db1b43dfbf4fd3c845d00f

Author: mgagne

manifests/plugins/ml2.pp

@@ -98,7 +98,8 @@
   $tunnel_id_ranges      = ['20:100'],
   $vxlan_group           = '224.0.0.1',
   $vni_ranges            = ['10:100'],
-  $enable_security_group = true
+  $enable_security_group = true,
+  $package_ensure        = 'present'
 ) {
 
   include neutron::params
@@ -120,7 +121,7 @@
   # Some platforms do not have a dedicated ml2 plugin package
   if $::neutron::params::ml2_server_package {
     package { 'neutron-plugin-ml2':
-      ensure => present,
+      ensure => $package_ensure,
       name   => $::neutron::params::ml2_server_package,
     }
     Package['neutron-plugin-ml2'] -> Neutron_plugin_ml2<||>

spec/classes/neutron_plugins_ml2_spec.rb

@@ -36,7 +36,8 @@
       :network_vlan_ranges   => ['10:50'],
       :tunnel_id_ranges      => ['20:100'],
       :vxlan_group           => '224.0.0.1',
-      :vni_ranges            => ['10:100'] }
+      :vni_ranges            => ['10:100'],
+      :package_ensure        => 'present' }
   end
 
   let :params do
@@ -71,7 +72,7 @@
       if platform_params.has_key?(:ml2_server_package)
         should contain_package('neutron-plugin-ml2').with(
           :name   => platform_params[:ml2_server_package],
-          :ensure => 'present'
+          :ensure => p[:package_ensure]
         )
         should contain_package('neutron-plugin-ml2').with_before(/Neutron_plugin_ml2\[.+\]/)
       end
@@ -167,6 +168,20 @@
         expect { subject }.to raise_error(Puppet::Error, /vni ranges are invalid./)
       end
     end
+
+    context 'when overriding package ensure state' do
+      before :each do
+        params.merge!(:package_ensure => 'latest')
+      end
+      it 'overrides package ensure state (if possible)' do
+        if platform_params.has_key?(:ml2_server_package)
+          should contain_package('neutron-plugin-ml2').with(
+            :name   => platform_params[:ml2_server_package],
+            :ensure => params[:package_ensure]
+          )
+        end
+      end
+    end
   end
 
   context 'on Debian platforms' do