Yuvipanda has submitted this change and it was merged.
Change subject: Revert "tools: store verbose logrotate logs"
......................................................................
Revert "tools: store verbose logrotate logs"
This reverts commit cfd163592c14df5000d9e18bf3a5ecfce50cee62.
Change-Id: Ib5a201ee47d8d64987369f0acb955d7ec987c5f2
---
D modules/toollabs/files/logrotate.crondaily
M modules/toollabs/manifests/init.pp
2 files changed, 0 insertions(+), 26 deletions(-)
Approvals:
Yuvipanda: Verified; Looks good to me, approved
diff --git a/modules/toollabs/files/logrotate.crondaily
b/modules/toollabs/files/logrotate.crondaily
deleted file mode 100644
index 964b2b2..0000000
--- a/modules/toollabs/files/logrotate.crondaily
+++ /dev/null
@@ -1,17 +0,0 @@
-#!/bin/sh
-# This file is managed by Puppet.
-# Based on logrotate-bundled cron file, Version: 3.8.7-1ubuntu1
-# Clean non existent log file entries from status file
-cd /var/lib/logrotate
-test -e status || touch status
-head -1 status > status.clean
-sed 's/"//g' status | while read logfile date
-do
- [ -e "$logfile" ] && echo "\"$logfile\" $date"
-done >> status.clean
-mv status.clean status
-
-test -x /usr/sbin/logrotate || exit 0
-# > instead of >> to clear logrotate.log daily
-# copy output to stderr on non-zero exit code to trigger a cron email
-/usr/sbin/logrotate -v /etc/logrotate.conf > /var/log/logrotate.log 2>&1 ||
cat /var/log/logrotate.log >&2
diff --git a/modules/toollabs/manifests/init.pp
b/modules/toollabs/manifests/init.pp
index ac780ff..19979fe 100644
--- a/modules/toollabs/manifests/init.pp
+++ b/modules/toollabs/manifests/init.pp
@@ -207,14 +207,5 @@
source => 'puppet:///modules/toollabs/40-tools-sudoers-no-warning',
}
- file { '/etc/cron.daily/logrotate':
- ensure => file,
- mode => '0664',
- owner => 'root',
- group => 'root',
- source => 'puppet:///modules/toollabs/logrotate.crondaily',
- requires => Package['logrotate']
- }
-
diamond::collector::localcrontab { 'localcrontabcollector': }
}
--
To view, visit https://gerrit.wikimedia.org/r/211890
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib5a201ee47d8d64987369f0acb955d7ec987c5f2
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Merlijn van Deen <valhall...@arctus.nl>
Gerrit-Reviewer: Yuvipanda <yuvipa...@gmail.com>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
