Hello Lars,

On Tue, Nov 20 2018, Lars Kruse wrote:
> Indeed this change was not well understood at that point in time and
> needed to be reversed. I just did this:
>  
> https://github.com/munin-monitoring/munin/commit/087478ed7df023733e8b3efb37703eadd3305791
> The change will be released with munin 2.0.44.

Awesome!

> While testing this issue, I stumbled upon another minor issue, that I
> fixed.  Maybe you would like to test the new state of the ipmi_sensor_
> plugin?
>
> curl 
> https://github.com/munin-monitoring/munin/raw/0b70ebf5ff/plugins/node.d/ipmi_sensor_.in
>  \
>   | sed 's#@@CONFDIR@@#/etc/munin#; s#@@PYTHON@@#/usr/bin/python3#'

I tested it on a host that's monitoring various temperatures, voltages,
and fans - works great.  I don't have any hardware that is mising the
"Assertions" output, so can't exactly test the code path in your change.

-- 
Gerald Turner <gtur...@unzane.com>        Encrypted mail preferred!
OpenPGP: 4096R / CA89 B27A 30FA 66C5 1B80  3858 EC94 2276 FDB8 716D

Attachment: signature.asc
Description: PGP signature

Reply via email to