ArielGlenn has submitted this change and it was merged.
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(-)
Approvals:
Ottomata: Looks good to me, but someone else must approve
ArielGlenn: Looks good to me, approved
jenkins-bot: Verified
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: merged
Gerrit-Change-Id: Ia72912583de5a25bd0b669e5a34ee0b31154e6e0
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: ArielGlenn <[email protected]>
Gerrit-Reviewer: ArielGlenn <[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
