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

Unified Diff: modules/nginx/files/logrotate

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/logrotate/manifests/init.pp ('k') | modules/nginx/manifests/init.pp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/nginx/files/logrotate
===================================================================
--- a/modules/nginx/files/logrotate
+++ b/modules/nginx/files/logrotate
@@ -1,10 +1,11 @@
-/srv/log/access.log /srv/log/error.log {
- daily
- rotate 30
- compress
- missingok
- nodateext
- postrotate
- /usr/bin/killall -HUP nginx
- endscript
+/var/log/nginx/access.log /var/log/nginx/error.log {
+ daily
+ rotate 30
+ compress
+ missingok
+ nodateext
+ sharedscripts
+ postrotate
+ [ ! -f /var/run/nginx.pid ] || kill -USR1 `cat /var/run/nginx.pid`
+ endscript
}
« no previous file with comments | « modules/logrotate/manifests/init.pp ('k') | modules/nginx/manifests/init.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld