Dzahn has submitted this change and it was merged.

Change subject: Apertium: Move logs to /srv/log
......................................................................


Apertium: Move logs to /srv/log

On SC(A|B), the /srv partition is much bigger than /, so move Apertium's
logs there.

Note: instead of hard-coding the path, this patch leverages the
configuration set for ::service::configuration (which for both SCA and
SCB is set to /srv/log).

Bug: T107900
Change-Id: I388f2392dd0d99b7a9e3252fec7575540d68937e
---
M modules/apertium/manifests/init.pp
1 file changed, 5 insertions(+), 3 deletions(-)

Approvals:
  KartikMistry: Looks good to me, but someone else must approve
  Filippo Giunchedi: Looks good to me, but someone else must approve
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/modules/apertium/manifests/init.pp 
b/modules/apertium/manifests/init.pp
index 02a509a..b400e2d 100644
--- a/modules/apertium/manifests/init.pp
+++ b/modules/apertium/manifests/init.pp
@@ -9,8 +9,6 @@
 # Number of APY instance processes to run
 # [*max_idle_seconds*]
 # Seconds to wait before shutdown idle process
-# [*log_dir*]
-# Place where Apertium-APY can put log files.
 # [*uid*]
 # The username apertium-apy will run with
 # [*gid*]
@@ -18,10 +16,14 @@
 class apertium(
     $num_of_processes = 1,
     $max_idle_seconds = 300,
-    $log_dir = '/var/log/apertium',
     $uid = 'apertium',
     $gid = 'apertium',
 ) {
+
+    include ::service::configuration
+
+    $log_dir = "${::service::configuration::log_dir}/apertium"
+
     package { [
         'apertium',
         'apertium-af-nl',

-- 
To view, visit https://gerrit.wikimedia.org/r/268852
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I388f2392dd0d99b7a9e3252fec7575540d68937e
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mobrovac <mobro...@wikimedia.org>
Gerrit-Reviewer: Alexandros Kosiaris <akosia...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Filippo Giunchedi <fgiunch...@wikimedia.org>
Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: KartikMistry <kartik.mis...@gmail.com>
Gerrit-Reviewer: Mobrovac <mobro...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to