search

lusis / chef-logstash

Semi-official Logstash cookbook
Other
271 stars 353 forks source link

Add logrotate maxsize parameter #456

Closed hartfordfive closed 7 years ago

hartfordfive commented 8 years ago

Currently, there is only a logrotate_size but no logrotate_max_size parameter for the logstash_instance resource. This can be an issue, for example, in a situation where a large volume of errors occur on the logstash instance. The end result is that the file can grow to consume all disk space before daily logrotate cron executes. Added support for this parameter prevents this from occurring.

martinb3 commented 7 years ago

Hi there! I've recently hopped in to help on this cookbook; if you'd like to rebase this against master, and have us review/merge it, please do so. If this PR is too out of date, and you think it should be closed, please do that as well. Thank you!

martinb3 commented 7 years ago

Rebased and merged in 1d20dbaa091d50ccaf0151bfbbc72639fa69c137.