Supervisor/supervisor — pull request #241 by jaraco (2013-05-25 23:14:43+00:00)
Title: Separate log targets for syslog (and other streams)

My last pull request added syslog support by adapting a patch supplied by a user to allow specifying 'syslog' as a special "filename" for the log targets. If specified, the setting would then direct the stream for that channel (stdout or stderr) to syslog.

Unfortunately, by invoking this functionality, Supervisord no longer can log to a file nor can it provide a copy of the log in the API, the latter of which was very important for our application.

This new patch, a series of changes that I worked over time and which we're now running in our production environment, allows the user to additionally configure two new settings, 'syslog_stdout' and 'syslog_stderr'. If either are indicated and True, that channel will be directed to syslog, independent of the setting for the file stream.

This change required substantial refactoring to implement as an additional, optional, stream, and I have taken care to describe those changes in deliberately small steps for easier review. All of the changes in this pull request were made as a bare minimum approach to achieve the stated goal in a robust way and no more.

Again, with tests passing and with this running in our production environment, I have confidence that it is a solid improvement. I hope you find the same. Please consider it for inclusion upstream.