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. 7, 2013, 3:06 p.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
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
}

Powered by Google App Engine
This is Rietveld