From 4a4c739b62bcffa65e7076bdf8881d4f1d393aa1 Mon Sep 17 00:00:00 2001
From: Abubakr-Sadik Nii Nai Davis <>
Date: Mon, 6 Nov 2023 15:24:53 +0000
Subject: [PATCH] (#1203) Fix lib directory permissions without changing the
 mode of files in the directory

This module changes the file mode of files in /usr/share/elasticsearch/lib everytime
Elasticsearch is upgraded with Apt. This is unnecessary and undesirable.

From this commit https://github.com/voxpupuli/puppet-elasticsearch/commit/854d9022b45e49c00f8b6f7e709a1f1b4372a005,
the intention was to fix the permission on the directory, but it is now
changing the file mode of all the files in the directory as well.

From tests it is alright to set the file mode on the directory to
0644 as Puppet does the right thing setting the directory mode to 0755,
irrespective of the umask value, while setting the file mode to 0644 as
expected.
---
 manifests/config.pp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/manifests/config.pp b/manifests/config.pp
index ecc45943d..5f48fb6e7 100644
--- a/manifests/config.pp
+++ b/manifests/config.pp
@@ -43,7 +43,7 @@
         ensure  => 'directory',
         group   => '0',
         owner   => 'root',
-        mode    => '0755',
+        mode    => '0644',
         recurse => true;
     }
     if $elasticsearch::manage_datadir {