Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request optionally lets the puppet module take care of log purging.
My typical use-case is to go back and investigate if there were any resource starvation bottle necks based on user reports or logging from the monitoring systems i.e. complaining on a slow web site. Short-lived resource starvation situations may cause significant problems with performance i.e. on a web server, but may go completely unnoticed on atop-logs with ten minutes interval, or munin graphs with 5 minutes interval, hence I'm typically setting the logging interval to 30s, sometimes even lower. With low log intervals the log files tend to take a lot of space, and with this use-case I typically don't need older logs than yesterdays log.
Some distributions tend to try to rotate the atop logs through logrotate, this sometimes causes problems, i.e. if the log file is rotated at 06:00 rather than midnight, this command will not work: "atop -r -b 04:30". I've also seen distributions where log rotation is done both by the init script and logrotate, that causes a mess. Hence if the purging is controlled by this puppet file, it also tries to remove any logrotation rules on atop, if possible.