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

Unified Diff: modules/nginx/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. 8, 2013, 8:19 a.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
« no previous file with comments | « modules/nginx/files/logrotate ('k') | modules/sitescripts/manifests/init.pp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/nginx/manifests/init.pp
===================================================================
--- a/modules/nginx/manifests/init.pp
+++ b/modules/nginx/manifests/init.pp
@@ -40,20 +40,20 @@ class nginx (
ensure => link,
require => File["/etc/nginx/sites-available/${file}"],
target => "/etc/nginx/sites-available/${file}",
notify => Service['nginx']
}
}
}
-# file {'/etc/logrotate.d/nginx':
-# source => 'nginx/logrotate',
-# require => File['/etc/nginx/nginx.conf']
-# }
+ file {'/etc/logrotate.d/nginx':
+ source => 'puppet:///modules/nginx/logrotate',
+ require => Package['nginx']
+ }
service {'nginx':
ensure => running,
enable => true,
restart => '/etc/init.d/nginx reload',
hasstatus => true,
require => File['/etc/nginx/nginx.conf']
}
« no previous file with comments | « modules/nginx/files/logrotate ('k') | modules/sitescripts/manifests/init.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld