Merge pull request #135 from petems/fix_debian_runfolder_issue

adding /var/run/redis for Debian based hosts
voxpupuli · Nov 30, 2016 · 7b9955f · 7b9955f
2 parents c2717f8 + a097c04
commit 7b9955f
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 0 deletions.
diff --git a/manifests/config.pp b/manifests/config.pp
@@ -115,6 +115,13 @@
         owner  => $::redis::config_owner,
       }
 
+      file { '/var/run/redis':
+        ensure => 'directory',
+        owner  => $::redis::config_owner,
+        group  => $::redis::config_group,
+        mode   => '0755',
+      }
+
       if $::redis::ulimit {
         augeas { 'redis ulimit' :
           context => '/files/etc/default/redis-server',

diff --git a/spec/classes/redis_spec.rb b/spec/classes/redis_spec.rb
@@ -29,6 +29,35 @@
             'hasstatus'  => 'true'
           )
         end
+
+        case facts[:operatingsystem]
+          when 'Debian'
+
+          context 'on Debian' do
+
+            it do
+              is_expected.to contain_file('/var/run/redis').with({
+                :ensure => 'directory',
+                :owner  => 'redis',
+                :group  => 'root',
+                :mode   => '0755',
+              })
+            end
+
+          end
+
+        when 'Ubuntu'
+
+          it do
+            is_expected.to contain_file('/var/run/redis').with({
+              :ensure => 'directory',
+              :owner  => 'redis',
+              :group  => 'root',
+              :mode   => '0755',
+            })
+          end
+
+        end
       end
 
       describe 'with parameter activerehashing' do