Add logrotate cronjob to all servers, don`t run logrotate unnecessarily on provisioning and use con…

modules/filterserver/manifests/init.pp

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':