diff options
| author |  Jan Wagner <waja@cyconet.org>
| 2013-09-30 17:55:22 +0200 |
|---|---|---|
| committer | Jan Wagner <waja@cyconet.org>
| 2014-07-20 23:52:25 +0200 |
| commit | 69b719aa2e3bd8dfd643d7cbe60e6e165d5279ea (patch) | |
| tree | 8428904d65cc748620318cbfac5e3063b4ddfbb4 | |
| parent | 71063655e558224d692950ac9abaa39e0b7c6585 (diff) | |
| download | monitoring-plugins-69b719aa2e3bd8dfd643d7cbe60e6e165d5279ea.tar.gz | |
check_ifstatus: perfdata bug
The perfdata output violates the current Nagios Plugin Development Guidelines
(http://nagiosplug.sourceforge.net/developer-guidelines.html#AEN202 :
"space separated list of label/value pairs").
Thus the Addon PNP did not read the perdata correctly.
The patch replaces the commas with spaces in the perfdata output.
Many thanks to Patric Wust.
---
Closes #1169 and #721.
| -rw-r--r-- | THANKS.in | 1 | ||||
| -rwxr-xr-x | plugins-scripts/check_ifstatus.pl | 2 |
2 files changed, 2 insertions, 1 deletions
@@ -323,3 +323,4 @@ Eric J. Mislivec Jean-Claude Computing Andy Brist Mikael Falkvidd +Patric Wust diff --git a/plugins-scripts/check_ifstatus.pl b/plugins-scripts/check_ifstatus.pl index fb17d983..709ad174 100755 --- a/plugins-scripts/check_ifstatus.pl +++ b/plugins-scripts/check_ifstatus.pl @@ -221,7 +221,7 @@ foreach $key (keys %ifStatus) { $ifexclude, $ifunused); } -my $perfdata = sprintf("up=%d,down=%d,dormant=%d,excluded=%d,unused=%d",$ifup,$ifdown,$ifdormant,$ifexclude,$ifunused); +my $perfdata = sprintf("up=%d down=%d dormant=%d excluded=%d unused=%d",$ifup,$ifdown,$ifdormant,$ifexclude,$ifunused); print ("$state: $answer |$perfdata\n"); exit $ERRORS{$state}; |