Faidon Liambotis has submitted this change and it was merged.

Change subject: Remove misc::maintenance::foundationwiki cronjobs
......................................................................


Remove misc::maintenance::foundationwiki cronjobs

db1008 is no more (NXDOMAIN) and this besides being noop, produced
confusing errors in production logs. Kill with fire.

Change-Id: Ia0617a147164f4153b63b3b281299b38f26f3dd3
---
M manifests/misc/maintenance.pp
M manifests/site.pp
2 files changed, 0 insertions(+), 32 deletions(-)

Approvals:
  Faidon Liambotis: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/manifests/misc/maintenance.pp b/manifests/misc/maintenance.pp
index a2623a6..861c46f 100644
--- a/manifests/misc/maintenance.pp
+++ b/manifests/misc/maintenance.pp
@@ -2,35 +2,6 @@
 
 # mw maintenance/batch hosts
 
-class misc::maintenance::foundationwiki( $enabled = false ) {
-
-       system::role { "misc::maintenance::foundationwiki": description => 
"Misc - Maintenance Server: foundationwiki" }
-
-       # This is broken and points to a non-existing DB
-       cron { 'updatedays':
-               user => apache,
-               minute => '*/15',
-               command => '/usr/local/bin/mwscript 
extensions/ContributionReporting/PopulateFundraisingStatistics.php 
foundationwiki --op updatedays > 
/tmp/PopulateFundraisingStatistics-updatedays.log',
-               ensure => $enabled ?{
-                       true => present,
-                       false => absent,
-                       default => absent
-               };
-       }
-
-       # This is broken and points to a non-existing DB
-       cron { 'populatefundraisers':
-               user => apache,
-               minute => 5,
-               command => '/usr/local/bin/mwscript 
extensions/ContributionReporting/PopulateFundraisingStatistics.php 
foundationwiki --op populatefundraisers > 
/tmp/PopulateFundraisingStatistics-populatefundraisers.log',
-               ensure => $enabled ?{
-                       true => present,
-                       false => absent,
-                       default => absent
-               };
-       }
-}
-
 class misc::maintenance::refreshlinks( $enabled = false ) {
 
        require mediawiki
diff --git a/manifests/site.pp b/manifests/site.pp
index 15a460e..7a93799 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -2750,9 +2750,6 @@
     # (bug 15434) Periodical run of currently disabled special pages
     # to be run against PMTPA slaves
     class { misc::maintenance::updatequerypages: enabled => true }
-
-    # totally broken, misconfigured in non-version reporting-setup.php
-    class { misc::maintenance::foundationwiki: enabled => false }
 }
 
 node /^testsearch100[1-3]\.eqiad\.wmnet/ {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia0617a147164f4153b63b3b281299b38f26f3dd3
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Reedy <re...@wikimedia.org>
Gerrit-Reviewer: Faidon Liambotis <fai...@wikimedia.org>
Gerrit-Reviewer: Jgreen <jgr...@wikimedia.org>
Gerrit-Reviewer: Mwalker <mwal...@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