sqelf container unstable. about seq-input-gelf HOT 7 CLOSED

Thanks for the follow-up. Gandelf is cool 🙂.

sqelf is a native Rust GELF server that's developed in this repo (not Ruby 😁). We kicked it off because we wanted to run the same code in a low-dependency Seq plug-in ([Seq.Input.Gelf}(https://nuget.org/packages/seq.input.gelf)), among other things.

Currently, the run.sh in the sqelf container's Dockerfile uses exec to swap out the initial bash process for sqelf - which won't necessarily cause the container to exit if the child seqcli process fails. We think we can improve on this by dropping the exec and setting the pipefail Bash option. Fingers crossed this will cause any process-level failures to propagate and get noticed by the host.

It's also possible that this issue is a hang or other kind of bug in sqelf or seqcli - if so, we'll be on the case and try to track it down.

Cheers,
Nick

from seq-input-gelf.

Thanks for the note, @svrooij - we'll take a look and let you know what we come up with.

from seq-input-gelf.

I just found a real nice bridge between gelf and seq (among others) https://github.com/jwulf/Gandelf but it had some small double encoding issue, that I fixed in my fork.

Your solution used some ruby gelf server that is pipes to your seqcli tool. The Gandelf solution uses a node Gelf-server and then does a call to the api. If the proces crashes it will automatically kill the container (which can then be restarted by docker auto restart).

from seq-input-gelf.

We've just published a new version of the Docker container as datalust/sqelf:1.0.205. This includes some changes to the run.sh script so that if either side of the pipe fails we'll exit the container with a non-zero code.

from seq-input-gelf.

I think we should close this and see how the newer version goes in the wild - please let us know if you decide to redeploy it, @svrooij - thanks!

from seq-input-gelf.

@nblumhardt I've redeployed it and it's running the logs of a few containers. Will let you guys know if the stability changed.

from seq-input-gelf.

Awesome, thanks @svrooij 👍

from seq-input-gelf.