Legoktm has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/259639

Change subject: extdist: Split skindist log into a separate file
......................................................................

extdist: Split skindist log into a separate file

The cron job runs can overlap, leading to combined log files, which are
annoying to read through.

Change-Id: I4bd5b5e81ad35b4d234ddd280dc861de00fdfd88
---
M modules/extdist/manifests/init.pp
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/39/259639/1

diff --git a/modules/extdist/manifests/init.pp 
b/modules/extdist/manifests/init.pp
index d43fb94..a3c6249 100644
--- a/modules/extdist/manifests/init.pp
+++ b/modules/extdist/manifests/init.pp
@@ -5,7 +5,7 @@
 #
 class extdist(
     $base_dir = '/srv',
-    $log_path = '/var/log/extdist'
+    $log_dir = '/var/log/'
 ){
     $dist_dir     = "${base_dir}/dist"
     $clone_dir    = "${base_dir}/extdist"
@@ -17,7 +17,7 @@
         'API_URL'   => 'https://www.mediawiki.org/w/api.php',
         'GIT_URL'   => 
'https://gerrit.wikimedia.org/r/mediawiki/extensions/%s',
         'DIST_PATH' => "${dist_dir}/extensions",
-        'LOG_FILE'  => $log_path,
+        'LOG_FILE'  => "${log_dir}/extdist",
         'SRC_PATH'  => $src_path,
         'PID_FILE'  => "${pid_folder}/pid.lock",
         'COMPOSER'  => "${composer_dir}/vendor/bin/composer"
@@ -27,7 +27,7 @@
         'API_URL'   => 'https://www.mediawiki.org/w/api.php',
         'DIST_PATH' => "${dist_dir}/skins",
         'GIT_URL'   => 'https://gerrit.wikimedia.org/r/mediawiki/skins/%s',
-        'LOG_FILE'  => $log_path,
+        'LOG_FILE'  => "${log_dir}/skindist",
         'SRC_PATH'  => $src_path,
         'PID_FILE'  => "${pid_folder}/skinpid.lock",
         'COMPOSER'  => "${composer_dir}/vendor/bin/composer"

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4bd5b5e81ad35b4d234ddd280dc861de00fdfd88
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Legoktm <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to