search

rsmuc / health_monitoring_plugins

Several plugins for icinga / nagios
GNU General Public License v2.0
16 stars 15 forks source link

Performance Data of check_snmp_raritan #15

Closed crsprsp closed 4 years ago

crsprsp commented 5 years ago

Hi,

when i try to query a Raritan PDU, i get the following output:

OK - Inlet. 4.173 A. 229.0 V. 818.0 W. 956.0 VA. 0.86. 427613.0 Wh. 49.9 Hz | 'Sensor 0'=4.173A;0.0:20.8;0.0:25.6;; 'Sensor 1'=229.0V;194.0:247.0;188.0:254.0;; 'Sensor 2'=818.0W;0.0:3250.0;0.0:3500.0;; 'Sensor 3'=956.0VA;0.0:0.0;0.0:0.0;; 'Sensor 4'=0.86;0.0:0.0;0.0:0.0;; 'Sensor 5'=427613.0Wh;0.0:0.0;0.0:0.0;; 'Sensor 6'=49.9Hz;0.0:0.0;0.0:0.0;; 4.173 A: normal 229.0 V: normal 818.0 W: normal 956.0 VA: normal 0.86 : normal 427613.0 Wh: normal 49.9 Hz: normal

The problem is now to get it in Graphite. There is only the 0.86 shown. All the others are not possible to write to Graphite. Seems like this is caused by the Units that are delivered with the Perfdata. Can this be switched off in a way?

bye, Rico

crsprsp commented 5 years ago

Found the following guideline for nagios:

https://www.monitoring-plugins.org/doc/guidelines.html#AEN201

UOM...value...

rsmuc commented 5 years ago

A, V, W etc. are not officially allowed UOMs. But pnp4nagios is happy with it. Maybe I could move the UOMs to the label.

crsprsp commented 5 years ago

oh, that would be so nice!

i get the following from the icinga service:

Nov 21 16:23:23 tux icinga2[25016]: Ignoring invalid perfdata value: 'Sensor 3'=955.0VA;0.0:0.0;0.0:0.0;; Context: (0) Processing check result for 'PDU1!PDU Health'... Nov 21 16:23:23 tux icinga2[25016]: Ignoring invalid perfdata value: 'Sensor 5'=430330.0Wh;0.0:0.0;0.0:0.0;; Context: (0) Processing check result for 'PDU1!PDU Health'... Nov 21 16:23:23 tux icinga2[25016]: Ignoring invalid perfdata value: 'Sensor 6'=50.0Hz;0.0:0.0;0.0:0.0;; Context: (0) Processing check result for 'PDU1!PDU Health'...

haxtibal commented 5 years ago

Icinga2's GraphiteWriter actually validates if a unit is one of the allowed ones (in contrast to Icinga 1 and pnp4nagios), and ignores it if not. Moving units to the label as @rsmuc suggests seems straight and clean for now.

It's a pity to loose explicit unit information anyway. Icinga 2 guys are aware of the problem (Icinga/icinga2#6071) and are considering a "new plugin API" as foundation to solve it. If we wanted the units back we should probably contribute there.

rsmuc commented 5 years ago

@crsprsp could you try the version from branch 0.1.0? that should fix your issue.

crsprsp commented 5 years ago

seems to work! very good!!!