Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/filterserver/manifests/init.pp

Issue 9351014: Add logrotate cronjob to all servers, don`t run logrotate unnecessarily on provisioning and use con… (Closed)
Patch Set: Created Feb. 7, 2013, 3:06 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: modules/filterserver/manifests/init.pp
===================================================================
--- a/modules/filterserver/manifests/init.pp
+++ b/modules/filterserver/manifests/init.pp
@@ -112,32 +112,16 @@ class filterserver {
}
file {'/etc/logrotate.d/nginx_easylist-downloads.adblockplus.org':
ensure => file,
require => Nginx::Hostconfig['easylist-downloads.adblockplus.org'],
source => 'puppet:///modules/filterserver/logrotate'
}
- exec {'ensure_logrotate_status':
- command => '/etc/cron.daily/logrotate',
- require => File['/etc/logrotate.d/nginx_easylist-downloads.adblockplus.org']
- }
-
- cron {'logrotate':
- ensure => present,
- require => [
- Exec['ensure_logrotate_status']
- ],
- command => '/usr/sbin/logrotate /etc/logrotate.conf',
- user => root,
- hour => '0',
- minute => '0'
- }
-
file {'/home/rsync/.ssh':
ensure => directory,
require => User['rsync'],
owner => rsync,
mode => 0600;
}
file {'/home/rsync/.ssh/known_hosts':

Powered by Google App Engine
This is Rietveld