Merge pull request #998 from tylerjl/logging-dir-remove-recursion

Remove recursive management from logdir

Author: tylerjl

.travis.yml

@@ -23,6 +23,9 @@ jobs:
     - TASK=beaker:debian-8-x64:snapshot
   - env:
     - TASK=beaker:debian-9-x64:snapshot
+  - env:
+    - OSS_PACKAGE=true
+    - TASK=beaker:debian-9-x64:snapshot
   include:
   - stage: intake
     env:

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## x.x.x (Month Day, Year)
 
 #### Fixes
+* The Elasticsearch log directory is no longer recursively managed to avoid stomping on the user/mode settings that Elasticsearch prefers.
 
 #### Features
 

manifests/config.pp

@@ -32,11 +32,10 @@
         group  => $elasticsearch::elasticsearch_group,
         owner  => $elasticsearch::elasticsearch_user;
       $elasticsearch::logdir:
-        ensure  => 'directory',
-        group   => $elasticsearch::elasticsearch_group,
-        owner   => $elasticsearch::elasticsearch_user,
-        mode    => '0750',
-        recurse => true;
+        ensure => 'directory',
+        group  => $elasticsearch::elasticsearch_group,
+        owner  => $elasticsearch::elasticsearch_user,
+        mode   => '0750';
       $elasticsearch::plugindir:
         ensure => 'directory',
         group  => $elasticsearch::elasticsearch_group,