Merge "Fix package_provider for RedHat systems"

manifests/messaging.pp

@@ -75,6 +75,7 @@ class cloud::messaging(
 
   # Packaging issue: https://bugzilla.redhat.com/show_bug.cgi?id=1033305
   if $::osfamily == 'RedHat' {
+    $package_provider = 'yum'
     file {'/usr/sbin/rabbitmq-plugins':
       ensure => link,
       target => '/usr/lib/rabbitmq/bin/rabbitmq-plugins'
@@ -85,6 +86,9 @@ class cloud::messaging(
       target => '/usr/lib/rabbitmq/bin/rabbitmq-env'
     }
   }
+  else {
+    $package_provider  = $rabbitmq::params::package_provider
+  }
 
   class { 'rabbitmq':
     delete_guest_user        => true,
@@ -95,6 +99,7 @@ class cloud::messaging(
     node_ip_address          => $rabbitmq_ip,
     port                     => $rabbitmq_port,
     erlang_cookie            => $erlang_cookie,
+    package_provider         => $package_provider,
   }
 
   rabbitmq_vhost { '/':

spec/classes/cloud_messaging_spec.rb

@@ -42,6 +42,7 @@ describe 'cloud::messaging' do
           :node_ip_address          => params[:rabbitmq_ip],
           :erlang_cookie            => params[:erlang_cookie],
           :port                     => '5672',
+          :package_provider         => platform_params[:package_provider],
         )
     end
 
@@ -96,6 +97,10 @@ describe 'cloud::messaging' do
       { :osfamily => 'Debian' }
     end
 
+    let :platform_params do
+      { :package_provider => 'apt' }
+    end
+
     it_configures 'openstack messaging'
   end
 
@@ -104,6 +109,11 @@ describe 'cloud::messaging' do
       { :osfamily => 'RedHat' }
     end
 
+
+    let :platform_params do
+      { :package_provider => 'yum' }
+    end
+
     let :params do
       { :erlang_cookie => 'MY_COOKIE' }
     end