Dzahn has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/341593 )

Change subject: Gerrit: Remove reviewer counts cron, nobody is using it
......................................................................


Gerrit: Remove reviewer counts cron, nobody is using it

Literally: no requests in last 30 days

Change-Id: Ie3439b9873deb4f2531258f34955e64ce2ff73d9
---
M modules/gerrit/manifests/crons.pp
1 file changed, 0 insertions(+), 16 deletions(-)

Approvals:
  Krinkle: Looks good to me, but someone else must approve
  Hashar: Looks good to me, but someone else must approve
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/modules/gerrit/manifests/crons.pp 
b/modules/gerrit/manifests/crons.pp
index 706c3af..0102be9 100644
--- a/modules/gerrit/manifests/crons.pp
+++ b/modules/gerrit/manifests/crons.pp
@@ -6,20 +6,4 @@
         user    => 'root',
         minute  => [0, 15, 30, 45],
     }
-
-    # This is useful information about the distribution of reviewers.
-    # Gerrit's rest api doesn't provide an easy way to get this data.
-    file { '/var/www/reviewer-counts.json':
-        ensure => 'present',
-        owner  => 'gerrit2',
-        group  => 'root',
-        mode   => '0644',
-    }
-
-    cron { 'list_reviewer_counts':
-        command => "/usr/bin/java -jar 
/var/lib/gerrit2/review_site/bin/gerrit.war gsql -d 
/var/lib/gerrit2/review_site/ --format JSON_SINGLE -c 'SELECT changes.change_id 
AS change_id, COUNT(DISTINCT patch_set_approvals.account_id) AS reviewer_count 
FROM changes LEFT JOIN patch_set_approvals ON (changes.change_id = 
patch_set_approvals.change_id) GROUP BY changes.change_id' > 
/var/www/reviewer-counts.json",
-        user    => 'gerrit2',
-        hour    => 1,
-        require => File['/var/www/reviewer-counts.json'],
-    }
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie3439b9873deb4f2531258f34955e64ce2ff73d9
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Chad <ch...@wikimedia.org>
Gerrit-Reviewer: Aklapper <aklap...@wikimedia.org>
Gerrit-Reviewer: Chad <ch...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: Hashar <has...@free.fr>
Gerrit-Reviewer: Jforrester <jforres...@wikimedia.org>
Gerrit-Reviewer: Krinkle <krinklem...@gmail.com>
Gerrit-Reviewer: MZMcBride <w...@mzmcbride.com>
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