Manage logging directory and file

Fixes voxpupuli#172
syseleven · Sep 20, 2017 · 628c1e0 · 628c1e0
1 parent a465734
commit 628c1e0
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/manifests/config.pp b/manifests/config.pp
@@ -18,6 +18,7 @@
 
   $logging          = $::icingaweb2::logging
   $logging_file     = $::icingaweb2::logging_file
+  $logging_dir      = dirname($::icingaweb2::logging_file)
   $logging_level    = $::icingaweb2::logging_level
   $preferences_type = 'ini'
   $show_stacktraces = $::icingaweb2::show_stacktraces
@@ -47,6 +48,19 @@
     path => $::path,
   }
 
+  file { $logging_dir:
+    ensure => directory,
+    owner  => $conf_user,
+    group  => 'adm',
+    mode   => '0750',
+  }
+  file { $logging_file:
+    ensure => file,
+    owner  => $conf_user,
+    group  => 'adm',
+    mode   => '0640',
+  }
+
   icingaweb2::inisection {'logging':
     target   => "${conf_dir}/config.ini",
     settings => {

diff --git a/spec/classes/config_spec.rb b/spec/classes/config_spec.rb
@@ -17,6 +17,14 @@
         it { is_expected.to contain_icingaweb2__inisection('logging') }
         it { is_expected.to contain_icingaweb2__inisection('global') }
         it { is_expected.to contain_icingaweb2__inisection('themes') }
+        it { is_expected.to contain_file('/var/log/icingaweb2')
+          .with_ensure('directory')
+          .with_mode('0750')
+        }
+        it { is_expected.to contain_file('/var/log/icingaweb2/icingaweb2.log')
+          .with_ensure('file')
+          .with_mode('0640')
+        }
       end
 
       context 'with import_schema => true and db_type => mysql' do