Gehel has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/345633 )
Change subject: Default /etc/elasticsearch to 0755
......................................................................
Default /etc/elasticsearch to 0755
The elasticsearch package seems to create this as 0750 which
is a bit annoying when just looking around to debug something.
There are no secrets here so lets prevent sudo-fatigue by making it
world readable.
Change-Id: Ib479e8d956c17c73bb20b9052ad8d46f379c1c0d
---
M modules/elasticsearch/manifests/init.pp
1 file changed, 10 insertions(+), 0 deletions(-)
Approvals:
jenkins-bot: Verified
Gehel: Looks good to me, approved
diff --git a/modules/elasticsearch/manifests/init.pp
b/modules/elasticsearch/manifests/init.pp
index 1b4b606..ff4f2c7 100644
--- a/modules/elasticsearch/manifests/init.pp
+++ b/modules/elasticsearch/manifests/init.pp
@@ -146,6 +146,16 @@
java_package => $java_package,
}
+ # Package defaults this to 0750, which is annoying
+ # for debugging. There are no secrets here so make
+ # visible.
+ file { '/etc/elasticsearch':
+ ensure => directory,
+ owner => 'root',
+ group => 'root',
+ mode => '0755',
+ }
+
file { '/etc/elasticsearch/elasticsearch.yml':
ensure => file,
owner => 'root',
--
To view, visit https://gerrit.wikimedia.org/r/345633
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib479e8d956c17c73bb20b9052ad8d46f379c1c0d
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: EBernhardson <[email protected]>
Gerrit-Reviewer: Gehel <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
