RobH has submitted this change and it was merged.

Change subject: [cirrus maint] redirect stderr to log and use full mwscript path
......................................................................


[cirrus maint] redirect stderr to log and use full mwscript path

The log files from this command are generated, but not populated with
anything. Likely they are empty because stderr was not also redirected
to the log files. Best guess for why nothing is running is that the full
path to mwscript should be use. If not the case it still doesn't hurt
to be explicit

Bug: T120843
Change-Id: I2247ae6b12b7dcf4b38d7104cbafa31960a0a6b3
---
M modules/mediawiki/manifests/maintenance/cirrussearch.pp
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Gehel: Looks good to me, but someone else must approve
  Alex Monk: Looks good to me, but someone else must approve
  RobH: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/mediawiki/manifests/maintenance/cirrussearch.pp 
b/modules/mediawiki/manifests/maintenance/cirrussearch.pp
index 41ff559..bb55fce 100644
--- a/modules/mediawiki/manifests/maintenance/cirrussearch.pp
+++ b/modules/mediawiki/manifests/maintenance/cirrussearch.pp
@@ -12,7 +12,7 @@
         minute   => 20,
         hour     => 0,
         weekday  => 2,
-        command  => '/usr/local/bin/expanddblist all | xargs -I{} -P 4 sh -c 
\'mwscript extensions/CirrusSearch/maintenance/updateSuggesterIndex.php 
--wiki={} --masterTimeout=10m --cluster=eqiad --optimize > 
/var/log/mediawiki/cirrus-suggest/{}.eqiad.log || true\''
+        command  => '/usr/local/bin/expanddblist all | xargs -I{} -P 4 sh -c 
\'/usr/local/bin/mwscript 
extensions/CirrusSearch/maintenance/updateSuggesterIndex.php --wiki={} 
--masterTimeout=10m --cluster=eqiad --optimize > 
/var/log/mediawiki/cirrus-suggest/{}.eqiad.log 2>&1 || true\''
     }
 
     cron { 'cirrus_build_completion_indices_codfw':
@@ -21,7 +21,7 @@
         minute   => 20,
         hour     => 0,
         weekday  => 2,
-        command  => '/usr/local/bin/expanddblist all | xargs -I{} -P 4 sh -c 
\'mwscript extensions/CirrusSearch/maintenance/updateSuggesterIndex.php 
--wiki={} --masterTimeout=10m --cluster=codfw --optimize > 
/var/log/mediawiki/cirrus-suggest/{}.codfw.log || true\''
+        command  => '/usr/local/bin/expanddblist all | xargs -I{} -P 4 sh -c 
\'/usr/local/bin/mwscript 
extensions/CirrusSearch/maintenance/updateSuggesterIndex.php --wiki={} 
--masterTimeout=10m --cluster=codfw --optimize > 
/var/log/mediawiki/cirrus-suggest/{}.codfw.log 2>&1 || true\''
     }
 
     file { '/var/log/mediawiki/cirrus-suggest':

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2247ae6b12b7dcf4b38d7104cbafa31960a0a6b3
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: EBernhardson <ebernhard...@wikimedia.org>
Gerrit-Reviewer: Alex Monk <kren...@gmail.com>
Gerrit-Reviewer: Gehel <guillaume.leder...@gmail.com>
Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: RobH <r...@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