opendev/puppet-zuul
Code Issues Proposed changes

Use WatchedFileHandler to avoid copytruncate.

Browse Source 
Python logging has a WatchedFileHandler which will do the correct thing
for logrotate when copytruncate is not used. Do this to avoid having
multiple copies of the same log in place. Also, this should avoid a race
where log data is lost between copying and truncating done by logrotate.

Change-Id: Idbbc7869004a03b8f19cfbd81ad07f5584ce1964
This commit is contained in:
Clark Boylan 2014-05-27 15:03:38 -07:00
parent 6e24605176
commit 0f585bef4f
2 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/merger.pp
View File

@ -39,7 +39,6 @@ class zuul::merger (
log => '/var/log/zuul/merger.log',
options => [
'compress',
'copytruncate',
'missingok',
'rotate 30',
'daily',
@ -51,7 +50,6 @@ class zuul::merger (
log => '/var/log/zuul/merger-debug.log',
options => [
'compress',
'copytruncate',
'missingok',
'rotate 30',
'daily',

3
manifests/server.pp
View File

@ -35,7 +35,6 @@ class zuul::server (
log => '/var/log/zuul/zuul.log',
options => [
'compress',
'copytruncate',
'missingok',
'rotate 30',
'daily',
@ -47,7 +46,6 @@ class zuul::server (
log => '/var/log/zuul/debug.log',
options => [
'compress',
'copytruncate',
'missingok',
'rotate 30',
'daily',
@ -59,7 +57,6 @@ class zuul::server (
log => '/var/log/zuul/gearman-server.log',
options => [
'compress',
'copytruncate',
'missingok',
'rotate 7',
'daily',