[collectd] Unsuported sensor by ipmi plugin

[Cyril Feraudet]

Le 2012-11-20 12:32, Cyril Feraudet a écrit :

> Hi all again,
>
> I've this in my log and no data about ipmi sensor in Collectd :
>
> ipmi plugin: sensor_list_add: Ignore sensor Power Supplies 
> power_supply
> (10.3), because I don't know how to handle its type (0x8, 
> power_supply).
> If you need this sensor, please file a bug report.
>
> It's about the power consumption in watt of a ProLiant DL380 G7 :
>
> # ipmitool sdr
> UID Light | 0 unspecified | ok
> Sys. Health LED | 0 unspecified | ok
> Power Supply 1 | 95 Watts | nc
> Power Supply 2 | 105 Watts | nc
> Power Supplies | 0 unspecified | nc
> Fan 1 | 43.12 unspecifi | nc
> Fan 2 | 43.12 unspecifi | nc
> Fan 3 | 41.55 unspecifi | nc
> Fan 4 | 49 unspecified | nc
> Fan 5 | 49 unspecified | nc
> Fan 6 | 13.72 unspecifi | nc
> Fans | 0 unspecified | nc
> Temp 1 | 19 degrees C | ok
> Temp 2 | 40 degrees C | ok
> Temp 3 | 40 degrees C | ok
> Temp 4 | 28 degrees C | ok
> ------
> Temp 29 | 35 degrees C | ok
> Temp 30 | 58 degrees C | ok
> Memory | 0 error | ok
> Power Meter | 226 Watts | cr <------------- this one
>
> To make it working I have added this below to ipmi.c, the message in
> logs disappear but no new graph in Collectd.
>
> Do you have an idea ?
>
> --- src/ipmi.c 2012-11-19 16:05:47.000000000 +0100
> +++ ../org/src/ipmi.c 2012-11-15 10:56:43.817949208 +0100
> @@ -302,10 +302,6 @@
> type = "voltage";
> break;
>
> - case IPMI_SENSOR_TYPE_POWER_SUPPLY:
> - type = "power";
> - break;
> -
> case IPMI_SENSOR_TYPE_CURRENT:
> type = "current";
> break;
>
> Regards,
>
> Cyril

Reply to myself : Adding IPMI_SENSOR_TYPE_POWER_SUPPLY case is what we 
need to handle this kind of sensors and ipmi plugin code is good.
Something is wrong in the OpenIPMI lib who doesn't permit to retrieve 
values from this kind of sensors by ipmi_sensor_id_get_reading();

If anyone know something about it ...

Cyril