search

fluent-plugins-nursery / fluent-plugin-systemd

This is a fluentd input plugin. It reads logs from the systemd journal.
Apache License 2.0
153 stars 43 forks source link

rescue JournalError #77

Closed mariusgrigoriu closed 3 years ago

mariusgrigoriu commented 5 years ago

I believe this is the fix to issue #70

According to stack traces move_next is raising an exception. This PR should rescue and allow the plugin to continue.

errm commented 5 years ago

Thanks for the contribution @mariusgrigoriu ...

Have you an example journal we could use to create a test case for this behaviour...

I am worried that this addition could result in continuously trying to call move next on a corrupted entry and not making any progress...

fadlinurhasan commented 3 years ago

Hi team,

We are facing the same issue, especially when migrating to ubuntu 20.04. Is there any plan to merge this PR?

thanks,

errm commented 3 years ago

Does this patch fix your issue @fadlinurhasan ?

fadlinurhasan commented 3 years ago

Yes @errm, so far it solves our issue.

errm commented 3 years ago

Thanks for the contibution, sorry it took so long to review and merge!