captain-coredump / weatherflow-udp

WeatherFlow Personal Weather Station driver for weewx, via UDP broadcast packets
GNU General Public License v3.0
54 stars 19 forks source link

WeatherFlowUDP logs to /weewxd not weewx #16

Open dalanbrown opened 3 years ago

dalanbrown commented 3 years ago

Looking in https://github.com/weewx/weewx/blob/master/bin/weeutil/log.py we see

def log_open(log_label='weewx'): syslog.openlog(log_label, syslog.LOG_PID | syslog.LOG_CONS)

Which is apparently overridden by --log-label command line options.

Looking in https://github.com/captain-coredump/weatherflow-udp/blob/master/bin/user/weatherflowudp.py we see

def logmsg(level, msg): syslog.syslog(level, 'weatherflowudp: %s: %s' % (threading.currentThread().getName(), msg))

So when you enable syslog filtering the weatherflowudp loggings says in syslog

e.g. Sep 9 00:00:30 weather64 /weewxd: weatherflowudp: MainThread: driver version is 1.10 Sep 9 00:00:30 weather64 /weewxd: weatherflowudp: MainThread: sensor map is {} Sep 9 00:00:30 weather64 /weewxd: weatherflowudp: MainThread: *** Sensor names per packet type

The recomendation for syslog is here: https://github.com/weewx/weewx/wiki/logging :programname,startswith,"wee" /var/log/weewx/weewx.log :programname,startswith,"wee" ~ :programname,isequal,"weewx-vantage" /var/log/weewx/vantage.log :programname,isequal,"weewx-vantage" ~ :programname,isequal,"weewx-rainwise" /var/log/weewx/rainwise.log :programname,isequal,"weewx-rainwise" ~

readme says:

log_raw_packets = False

Enable writing all raw UDP packets received to syslog,
(**or wherever weewx is configured to send log info**).  Will
fill up your logs pretty quickly, so only use it as
a debugging tool or to identify sensors.

So either the readme needs go be updated to offer up a change to /etc/rsyslog.d/10-weewx.conf or the log location ought to be changed o whatever the parent "--log-label" command line argument suggests or default to "weewx".

Workaround of course it to look in syslog but it took me a few minutes to figure out they had not been redirected.

jjvdgeer commented 3 years ago

It seems tkeffers branch has this fixed. You could try that one. Or mine, which has a bunch of other added features and is based on the tkeffer branch.