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. 7, 2013, 3:06 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« modules/logrotate/manifests/init.pp ('K')
|
« modules/nginx/files/logrotate ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | 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 => File['/etc/nginx/nginx.conf'] |
|
Felix Dahlke
2013/02/07 15:38:41
Why require the file here? Either the package, if
|
| + } |
| service {'nginx': |
| ensure => running, |
| enable => true, |
| restart => '/etc/init.d/nginx reload', |
| hasstatus => true, |
| require => File['/etc/nginx/nginx.conf'] |
| } |
« modules/logrotate/manifests/init.pp ('K')
|
« modules/nginx/files/logrotate ('k')
|
no next file »
|
no next file with comments »
