ArielGlenn has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/277214

Change subject: limit find to type file for stats log cleanup
......................................................................

limit find to type file for stats log cleanup

This will get rid of the emails from stat1002 with content:
rm: cannot remove /a/log/webrequest/archive/5xx-bits: Is a directory

Change-Id: Ia72912583de5a25bd0b669e5a34ee0b31154e6e0
---
M modules/statistics/manifests/rsync_job.pp
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/14/277214/1

diff --git a/modules/statistics/manifests/rsync_job.pp 
b/modules/statistics/manifests/rsync_job.pp
index 89356eb..9d4e28f 100644
--- a/modules/statistics/manifests/rsync_job.pp
+++ b/modules/statistics/manifests/rsync_job.pp
@@ -44,7 +44,7 @@
 
     cron { "prune_old_${name}_logs":
         ensure  => $prune_old_logs_ensure,
-        command => "/usr/bin/find ${destination} -mtime +${retention_days} 
-exec rm {} \\;",
+        command => "/usr/bin/find ${destination} -type f -mtime 
+${retention_days} -exec rm {} \\;",
         user    => $::statistics::user::username,
         minute  => 0,
         hour    => 9,

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia72912583de5a25bd0b669e5a34ee0b31154e6e0
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: ArielGlenn <[email protected]>

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

Reply via email to