]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Fixes Facts string comparisons in CentOS 7
authorGael Chamoulaud <gchamoul@redhat.com>
Wed, 9 Jul 2014 09:52:25 +0000 (11:52 +0200)
committerGael Chamoulaud <gchamoul@redhat.com>
Wed, 9 Jul 2014 17:43:13 +0000 (19:43 +0200)
- CentOS Project has adopted a new rule for versioning numbers. The major number
  matches the RHEL major number, but the minor number is generated from the
  release date. For example, CentOS 7.0.1407.
- Adds Support for Scientific Linux.

Change-Id: I3220206aa01b9f5c838e524b3fec723c92435153
Closes-Bug: 1338777
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
spec/classes/ceilometer_agent_notification_spec.rb

index a7ad76ebd88eee162ca820ceb091c154a020635e..f5bfe18e6a1d142e30a5d6c01cb0b7efe4f475a9 100644 (file)
@@ -84,9 +84,9 @@ describe 'ceilometer::agent::notification' do
 
   context 'on RHEL 7' do
     let :facts do
-      { :osfamily => 'RedHat',
-        :operatingsystem => 'RedHat',
-        :operatingsystemrelease => 7
+      { :osfamily                  => 'RedHat',
+        :operatingsystem           => 'RedHat',
+        :operatingsystemmajrelease => 7
       }
     end
 
@@ -100,9 +100,25 @@ describe 'ceilometer::agent::notification' do
 
   context 'on CentOS 7' do
     let :facts do
-      { :osfamily => 'RedHat',
-        :operatingsystem => 'CentOS',
-        :operatingsystemrelease => 7
+      { :osfamily                  => 'RedHat',
+        :operatingsystem           => 'CentOS',
+        :operatingsystemmajrelease => 7
+      }
+    end
+
+    let :platform_params do
+      { :agent_notification_package_name => 'openstack-ceilometer-collector',
+        :agent_notification_service_name => 'openstack-ceilometer-notification' }
+    end
+
+    it_configures 'ceilometer-agent-notification'
+  end
+
+  context 'on Scientific 7' do
+    let :facts do
+      { :osfamily                  => 'RedHat',
+        :operatingsystem           => 'Scientific',
+        :operatingsystemmajrelease => 7
       }
     end
 
@@ -116,8 +132,8 @@ describe 'ceilometer::agent::notification' do
 
   context 'on Fedora 20' do
     let :facts do
-      { :osfamily => 'RedHat',
-        :operatingsystem => 'Fedora',
+      { :osfamily               => 'RedHat',
+        :operatingsystem        => 'Fedora',
         :operatingsystemrelease => 20
       }
     end