[collectd] modbus plugin spamming console with debug info

[Eric Sandeen]

On 2/25/16 4:49 AM, Marc Fournier wrote:
> 
> Hello Eric !
> 
> Eric Sandeen <sandeen at sandeen.net> writes:
> 
>> I hadn't noticed this before, but after running on a systemd system it
>> comes to the forefront:
>>
>>> Message from syslogd at sandeen at Feb 24 22:39:48 ...
>>>  collectd:<21><00><00><00><05><01><04><02><00><02>
>>
>>> Broadcast message from systemd-journald at sandeen.net (Wed 2016-02-24 22:39:48 CST):
>>
>>> collectd[22528]: <22><00><00><00><05><01><04><02><00><19>
>>
>> on every modbus communication.  ;)
>>
>> The plugin sets modbus_set_debug(ctx, 1); - that seems a bit odd to do unconditionally.
>>
>> Has anyone else run into this?  Setting to debug should almost certainly be optional,
>> AFAICT.
> 
> Thanks for pointing this out !
> 
> Here's a patch which should prevent this:
> https://github.com/collectd/collectd/pull/1561
> 
> Does it sound an acceptable solution ?

Yeah, makes sense to me; I don't know why systemd elevates this to a broadcast
message; didn't happen on my older distros.  But the solution looks good.  Sorry
for not just sending a patch.  :)

-Eric