Am 29.08.2016 um 17:51 schrieb Matt Zahorik: > On Fri, Aug 12, 2016 at 5:06 PM, Michael Biebl <[email protected]> wrote: >> >> Am 05.08.2016 um 17:52 schrieb Matt Zahorik: >>> Package: rsyslog >>> Version: 8.16.0-1~bpo8+1 >>> Tags: patch >>> >>> Please find a diff to add the redis output module to rsyslog. >> >> [..] >> >>> It requires libhiredis at runtime. I'm not as familiar with the deb >>> packaging system, so I didn't include changes for that. The code >>> doesn't specify a minimum version. I successfully compiled and ran >>> against 0.11.0-4 included in jessie. >> >> Have you tested the plugin and do you use it regularly? > > I do use it regularly in my lab environment, about ~20 hosts. It has > been running without issue for several weeks. > >> Do you consider it production ready? > > After testing, I would say "no". It very stable while running, > scaling up to 200K messages per second with ease, but if redis > crashes, rsyslog also crashes. > > The code appears to have reconnect logic, but it doesn't properly free > memory or close the stale redis connection. > > There is a pull request to properly handle reconnections, and I will > be waiting to test this before deploying to production. > > https://github.com/rsyslog/rsyslog/pull/1109
Thanks for reporting back and your feedback. I'm bit reluctant to ship code with known issues as I don't want to burn our users. I guess I'll wait until this fix lands upstream before merging your patch. Regards, Michael -- Why is it that all of the instruments seeking intelligent life in the universe are pointed away from Earth?
signature.asc
Description: OpenPGP digital signature
