Ottomata has submitted this change and it was merged.

Change subject: Add --percent-lost flag to refinery data check
......................................................................


Add --percent-lost flag to refinery data check

Adds the --percent-lost argument to the analytics refinery role
when calling refinery-dump-status-webrequest-partitions. This will
make the script output percent of data lost instead of 'X'.

Also adds all Analytics' engineers as recepients of the daily email.

Bug: T113255
Change-Id: Idd058bc83ee1a21cae2ce390211c0162105ebe57
---
M manifests/role/analytics/refinery.pp
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Ottomata: Verified; Looks good to me, approved
  Joal: Looks good to me, but someone else must approve



diff --git a/manifests/role/analytics/refinery.pp 
b/manifests/role/analytics/refinery.pp
index d386fec..78283ed 100644
--- a/manifests/role/analytics/refinery.pp
+++ b/manifests/role/analytics/refinery.pp
@@ -199,14 +199,14 @@
     # I am not sure why this is not working.
     $hdfs_mount_point = '/mnt/hdfs'
 
-    $mail_to = '[email protected],[email protected]'
+    $mail_to = 
'[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]'
 
     # Since the 'stats' user is not in ldap, it is unnecessarily hard
     # to grant it access to the private data in hdfs. As discussed in
     #   https://gerrit.wikimedia.org/r/#/c/186254
     # the cron runs as hdfs instead.
     cron { 'refinery data check hdfs_mount':
-        command     => 
"${::role::analytics::refinery::path}/bin/refinery-dump-status-webrequest-partitions
 --hdfs-mount ${hdfs_mount_point} --datasets webrequest,raw_webrequest --quiet",
+        command     => 
"${::role::analytics::refinery::path}/bin/refinery-dump-status-webrequest-partitions
 --hdfs-mount ${hdfs_mount_point} --datasets webrequest,raw_webrequest --quiet 
--percent-lost",
         environment => "MAILTO=${$mail_to}",
         user        => 'hdfs',
         hour        => 10,

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idd058bc83ee1a21cae2ce390211c0162105ebe57
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mforns <[email protected]>
Gerrit-Reviewer: Joal <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to