FWIW, I have been using phusion/baseimage <https://phusion.github.io/baseimage-docker/> as a base for my Docker builds and it pipes syslog to stdout by default - that works fine for me.
That said, I think it would be better if weewx were to log to stdout by default since it is kind of the default for Unix/variants. Not depending on syslogd seems like a nice simplification of the runtime. It sounds like with the workaround I would be able to override the syslogd in weewx.conf which would be fine. Not ivory tower perfect but I understand the user impact. On Tuesday, May 19, 2020 at 8:05:09 PM UTC-4, Tom Keffer wrote: > > What I had in mind was to trim the syslog handler out of the default > weeutil.logger.LOGGING_STR, and, instead, put it in weewx.conf. You'd end > up in the same place, but not without risk. The problem is that the > installer would have to patch old weewx.conf files, inserting the syslog > handler. > > If this is a rare instance, and it does seem to be, then I'd prefer my > workaround. I'd rather explain to Docker users who, by and large, are > sophisticated, how to do this, than have the system log inexplicably > disappear for less experienced users. > > On Tue, May 19, 2020 at 3:55 PM Vince Skahan <[email protected] > <javascript:>> wrote: > >> On Tuesday, May 19, 2020 at 11:52:42 AM UTC-7, Tom Keffer wrote: >>> >>> Fair enough, but before I go implementing this... did it work? Did my >>> workaround disable syslog thoroughly enough that you could get started? >>> >>>> >>>> >> update - I got centos7 python2 setup.py to build/install/run ok by >> 'replacing' logger.py in the sources with a patched variant (using your >> code snippet) before doing the setup.py build+install. The only downside >> in brief testing is the container doesn't log anything, as there is no >> logging subsystem in the distributed centos7 base Docker image. >> >> Weewx 'does' run fine however. >> >> Perhaps we might ask the other Docker users and the usual developer folks >> for their thoughts ? >> >> - one option is switch the logging for everybody, but I don't know >> what they does for default installs, VMs, etc. as a result >> - another option is to just require superseding logger.py for >> RedHat(ish) Docker users only, and note it somehow in the docs (?) >> >> Updated repo is at https://github.com/vinceskahan/weewx-docker if >> anybody wants to take a look. Centos7 build Dockerfile and patched >> logger.py is under the 'build' directory. Dockerfile should be pretty >> obvious. >> >> Maybe get some more eyeballs on this one before messing with the core >> weewx quite yet ? >> >> -- >> You received this message because you are subscribed to the Google Groups >> "weewx-development" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected] <javascript:>. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/weewx-development/1811cab3-d5e8-40d5-a036-bcf5a07d81a0%40googlegroups.com >> >> <https://groups.google.com/d/msgid/weewx-development/1811cab3-d5e8-40d5-a036-bcf5a07d81a0%40googlegroups.com?utm_medium=email&utm_source=footer> >> . >> > -- You received this message because you are subscribed to the Google Groups "weewx-development" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/weewx-development/26fd80ce-1bf6-479b-9a43-8f5c8cc26393%40googlegroups.com.
