-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21806/#review43710
-----------------------------------------------------------


I thought I had fixed this awhile back, but it was from the other direction. 
Conntrackd no longer restarts on log rotate unless conntrackd was already 
running (6b7f91d7). I suppose this wouldn't be bad to put in as well, so long 
as we know that conntrackd will in fact get started when we want it (redundant 
virtual router). Another issue contributing to it was that we had split up the 
tiny disk into many partitions, leaving only a few hundred MB available and 
stranded across many parts of the file tree, since there are times when we want 
conntrackd we should probably have a system vm template that can handle its 
logfiles.

You need to create a bug report that we can tie this to.

- Marcus Sorensen


On May 22, 2014, 11:42 a.m., Joris van Lieshout wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21806/
> -----------------------------------------------------------
> 
> (Updated May 22, 2014, 11:42 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, daan Hoogland, Rajesh 
> Battala, Rohit Yadav, and Hugo Trippaers.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Conntrackd package has a bug where the comment in the default config file 
> states that stats logging is disabled by default but the config parameter is 
> set to on. The consequence for ACS is that a conntrackd-stats.log file is 
> created during the build of the svm. This logfile gets rotated by logrotate 
> which has a post action to restart conntrackd. Even if the svm is not a 
> redundant router. On vpc routers for instance the stats logging file can grow 
> quickly and fill up the /var volume killing the vm.
> 
> 
> Diffs
> -----
> 
>   tools/appliance/definitions/systemvm64template/postinstall.sh cc8ead9 
>   tools/appliance/definitions/systemvmtemplate/postinstall.sh 23e66dd 
> 
> Diff: https://reviews.apache.org/r/21806/diff/
> 
> 
> Testing
> -------
> 
> We are running this fix in our production environment for a couple months 
> allready.
> 
> 
> Thanks,
> 
> Joris van Lieshout
> 
>

Reply via email to