Giuseppe Lavagetto has submitted this change and it was merged.

Change subject: maintenance: actually ensure updatequerypages crons
......................................................................


maintenance: actually ensure updatequerypages crons

Change-Id: Ia6de6d025686214c6ac7c75e57dce411333893ad
---
M hieradata/hosts/mw1152.yaml
M hieradata/hosts/terbium.yaml
M modules/mediawiki/manifests/maintenance/updatequerypages.pp
3 files changed, 4 insertions(+), 3 deletions(-)

Approvals:
  Giuseppe Lavagetto: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/hieradata/hosts/mw1152.yaml b/hieradata/hosts/mw1152.yaml
index 984b8c7..a591e8e 100644
--- a/hieradata/hosts/mw1152.yaml
+++ b/hieradata/hosts/mw1152.yaml
@@ -23,3 +23,4 @@
 mediawiki::maintenance::purge_checkuser::ensure: absent
 mediawiki::maintenance::purge_securepoll::ensure: absent
 mediawiki::maintenance::jobqueue_stats::ensure: absent
+mediawiki::maintenance::updatequerypages::ensure: absent 
diff --git a/hieradata/hosts/terbium.yaml b/hieradata/hosts/terbium.yaml
index 1d553af..d7239e0 100644
--- a/hieradata/hosts/terbium.yaml
+++ b/hieradata/hosts/terbium.yaml
@@ -23,3 +23,4 @@
 mediawiki::maintenance::purge_checkuser::ensure: present
 mediawiki::maintenance::purge_securepoll::ensure: present
 mediawiki::maintenance::jobqueue_stats::ensure: present
+mediawiki::maintenance::updatequerypages::ensure: present
diff --git a/modules/mediawiki/manifests/maintenance/updatequerypages.pp 
b/modules/mediawiki/manifests/maintenance/updatequerypages.pp
index 730148e..5dbdd94 100644
--- a/modules/mediawiki/manifests/maintenance/updatequerypages.pp
+++ b/modules/mediawiki/manifests/maintenance/updatequerypages.pp
@@ -52,8 +52,8 @@
                 }
         }
 
-        define updatequerypages::enwiki::cronjob() {
-            $ensure = $mediawiki::maintenance::updatequerypages::status
+        define updatequerypages::enwiki::cronjob($ensure = 
$mediawiki::maintenance::updatequerypages::ensure) {
+
 
             Cron {
                 ensure => $ensure,
@@ -103,4 +103,3 @@
         updatequerypages::cronjob { ['s1@11', 's2@12', 's3@13', 's4@14', 
's5@15', 's6@16', 's7@17', 'silver@18']: }
         updatequerypages::enwiki::cronjob { ['updatequerypages-enwiki-only']: }
 }
-

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia6de6d025686214c6ac7c75e57dce411333893ad
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to