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

Delta Between Two Patch Sets: modules/logrotate/manifests/init.pp

Issue 29378979: Ticket70 - Remove duplicate/unnecessary logrotate cron job (Closed)
Left Patch Set: Created March 8, 2017, 11:46 p.m.
Right 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:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « modules/adblockplus/templates/crontab.erb ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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':
9 ensure => 'absent',
10 }
11
8 $config = hiera('logrotate::config', {}) 12 $config = hiera('logrotate::config', {})
9 create_resources('logrotate::config', $config) 13 create_resources('logrotate::config', $config)
10 } 14 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld