Better support for RedHat smartmontools 6.x.

Fixing Fedora support similar to #36.
jhoblitt · Sep 2, 2014 · bbcae38 · bbcae38
1 parent 71acbcc
commit bbcae38
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 6 deletions.
diff --git a/manifests/params.pp b/manifests/params.pp
@@ -36,7 +36,11 @@
     }
     'RedHat': {
       $config_file = $::operatingsystem ? {
-        'Fedora' => '/etc/smartd.conf',
+        'Fedora' => $::operatingsystemrelease ? {
+          # No, I am not going to support versions 1-9.
+          /10|11|12|13|14|15|16|17|18/ => '/etc/smartd.conf',
+          default                      => '/etc/smartmontools/smartd.conf',
+        },
         default  => $::operatingsystemmajrelease ? {
           '7'     => '/etc/smartmontools/smartd.conf',
           default => '/etc/smartd.conf',

diff --git a/spec/classes/smartd_spec.rb b/spec/classes/smartd_spec.rb
@@ -66,12 +66,23 @@
       end
 
       describe 'for operatingsystem Fedora' do
-        let(:facts) {{ :osfamily => 'RedHat', :operatingsystem => 'Fedora', :smartmontools_version => '5.43' }}
+        describe 'for operatingsystemrelease 18' do
+          let(:facts) {{ :osfamily => 'RedHat', :operatingsystem => 'Fedora', :operatingsystemrelease => '18', :smartmontools_version => '5.43' }}
 
-        it_behaves_like 'default', {}
-        it { should_not contain_shell_config('start_smartd') }
-        it { should contain_service('smartd').with_ensure('running').with_enable(true) }
-        it { should contain_file('/etc/smartd.conf').with_notify('Service[smartd]') }
+          it_behaves_like 'default', {}
+          it { should_not contain_shell_config('start_smartd') }
+          it { should contain_service('smartd').with_ensure('running').with_enable(true) }
+          it { should contain_file('/etc/smartd.conf').with_notify('Service[smartd]') }
+        end
+
+        describe 'for operatingsystemrelease 19' do
+          let(:facts) {{ :osfamily => 'RedHat', :operatingsystem => 'Fedora', :operatingsystemrelease => '19', :smartmontools_version => '6.1' }}
+
+          it_behaves_like 'default', { :config_file => '/etc/smartmontools/smartd.conf' }
+          it { should_not contain_shell_config('start_smartd') }
+          it { should contain_service('smartd').with_ensure('running').with_enable(true) }
+          it { should contain_file('/etc/smartmontools/smartd.conf').with_notify('Service[smartd]') }
+        end
       end
     end