Merge "Fixes OVS agent pkg and service name"

manifests/params.pp

@@ -95,8 +95,16 @@ class neutron::params {
       $ml2_server_package = false
     }
 
-    $ovs_agent_package   = 'neutron-plugin-openvswitch-agent'
+    case $::os_package_type {
-    $ovs_agent_service   = 'neutron-plugin-openvswitch-agent'
+      'debian': {
+        $ovs_agent_package   = 'neutron-openvswitch-agent'
+        $ovs_agent_service   = 'neutron-openvswitch-agent'
+      }
+      default: {
+        $ovs_agent_package   = 'neutron-plugin-openvswitch-agent'
+        $ovs_agent_service   = 'neutron-plugin-openvswitch-agent'
+      }
+    }
     $ovs_server_package  = 'neutron-plugin-openvswitch'
     $ovs_cleanup_service = false
     $ovs_package         = 'openvswitch-switch'

spec/classes/neutron_agents_ml2_ovs_spec.rb

@@ -244,7 +244,24 @@ describe 'neutron::agents::ml2::ovs' do
   context 'on Debian platforms' do
     let :facts do
       @default_facts.merge(test_facts.merge({
-         :osfamily => 'Debian'
+         :osfamily => 'Debian',
+         :os_package_type => 'debian'
+      }))
+    end
+
+    let :platform_params do
+      { :ovs_agent_package => 'neutron-openvswitch-agent',
+        :ovs_agent_service => 'neutron-openvswitch-agent' }
+    end
+
+    it_configures 'neutron plugin ovs agent with ml2 plugin'
+  end
+
+  context 'on Ubuntu platforms' do
+    let :facts do
+      @default_facts.merge(test_facts.merge({
+         :osfamily => 'Debian',
+         :os_package_type => 'ubuntu'
       }))
     end