On Mon, May 21, 2012 at 7:54 PM, Daniel Pocock <dan...@pocock.com.au> wrote: > b) data is likely to be missed if it is logged between the time > update.sh finishes and the rotation of a file for any particular vhost > > Why does the README insist on prerotate and not use postrotate?
Probably, because it's easy. > I've discovered that after rotation, logrotate can give the rotated > filename to the postrotate script, and using nosharedscripts, logrotate > can call awstats multiple times, once for each vhost, just as it > finishes the rotation of that host I'm not sure if reloading Apache per every vhost is a good idea. Why this can't be done once? > All that is needed is some wrapper script around awstats to select the > correct domain based on the path in $1 and pass the -config option to > awstats too > > Does this address all the issues raised by contributors to this bug report? Partially. See above: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=590074#70 On Tue, May 22, 2012 at 12:52 AM, Daniel Pocock <dan...@pocock.com.au> wrote: > I've contributed a script for fixing this, it is commit c0482b4109176e05 > on master > > It is based on Sergey's update.sh, but it does each log file separately > just after rotation For next release I've reverted the above commits. I don't like idea of code duplication. Can you consider to add needed code to update.sh? -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
