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

Side by Side Diff: modules/logrotate/manifests/init.pp

Issue 29378979: Ticket70 - Remove duplicate/unnecessary logrotate cron job (Closed)
Patch Set: For comment 2 Created March 9, 2017, 4:44 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « modules/adblockplus/templates/crontab.erb ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 class logrotate { 1 class logrotate {
2 exec {'ensure_logrotate_status': 2 exec {'ensure_logrotate_status':
3 command => '/etc/cron.daily/logrotate', 3 command => '/etc/cron.daily/logrotate',
4 path => ["/usr/bin/", "/bin/"], 4 path => ["/usr/bin/", "/bin/"],
5 onlyif => 'test ! -f /var/lib/logrotate/status' 5 onlyif => 'test ! -f /var/lib/logrotate/status'
6 } 6 }
7 7
8 cron {'logrotate': 8 cron {'logrotate':
9 ensure => present, 9 ensure => 'absent',
10 require => Exec['ensure_logrotate_status'],
11 command => '/usr/sbin/logrotate /etc/logrotate.conf',
12 environment => hiera('cron::environment', []),
13 user => root,
14 hour => '0',
15 minute => '0'
16 } 10 }
17 11
18 $config = hiera('logrotate::config', {}) 12 $config = hiera('logrotate::config', {})
19 create_resources('logrotate::config', $config) 13 create_resources('logrotate::config', $config)
20 } 14 }
OLDNEW
« no previous file with comments | « modules/adblockplus/templates/crontab.erb ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld