Gehel has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/341297 )
Change subject: elasticsearch init $data_dir creation requires installed package
......................................................................
elasticsearch init $data_dir creation requires installed package
The installed package creates the user that is needed
here.
Change-Id: Id05f750722f622b1cff79e117abeac11b8574520
---
M modules/elasticsearch/manifests/init.pp
1 file changed, 5 insertions(+), 4 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 bedba0f..4e1c18a 100644
--- a/modules/elasticsearch/manifests/init.pp
+++ b/modules/elasticsearch/manifests/init.pp
@@ -231,10 +231,11 @@
source => 'puppet:///modules/elasticsearch/logrotate',
}
file { $data_dir:
- ensure => directory,
- owner => 'elasticsearch',
- group => 'elasticsearch',
- mode => '0755',
+ ensure => directory,
+ owner => 'elasticsearch',
+ group => 'elasticsearch',
+ mode => '0755',
+ require => Package['elasticsearch'],
}
# GC logs rotation is done by the JVM, but on JVM restart, the logs left by
# the previous instance are left alone. This cron takes care of cleaning up
--
To view, visit https://gerrit.wikimedia.org/r/341297
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id05f750722f622b1cff79e117abeac11b8574520
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Gehel <guillaume.leder...@wikimedia.org>
Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
