Discussion :

[musha76]

Bonjour,

J'ai crée un plugin nagios pour surveiller mes ressources informatique, cependant j'ai l'impression que j'ai un problème avec mon script que voici :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
#!/bin/bash
 
#Arguments
while getopts A:B:C:D:E:F:hV OPT
do
   case $OPT in
        A)      AUTOMATE=$OPTARG ;;
        B)      OBJECT=$OPTARG ;;
        C)      IDENTIFIER=$OPTARG ;;
        D)      WARN=$OPTARG ;;
        E)      CRIT=$OPTARG ;;
        F)      MAX=$OPTARG ;;
        h)
          print_help
          exit $STATE_UNKNOWN
          ;;
        V)
          print_version
          exit $STATE_UNKNOWN
          ;;
    esac
done
 
 
#On va chercher la valeur
val=$(/tmp/bacnet-stack-0.8.3/bin/bacrp $AUTOMATE $OBJECT $IDENTIFIER 85 | sed 's/\r//')
unite=$(/tmp/bacnet-stack-0.8.3/bin/bacrp $AUTOMATE $OBJECT $IDENTIFIER 117 | sed 's/\r//')
 
 
#Arrondir le nombre
valeur=$(echo $val | cut -f1 -d\.)
 
 
#Réinscription de l'unité
unity=$(echo $unite | sed 's/degrees-celsius/°C/g' | sed 's/percent-relative-humidity/% humidité relative/g' | sed 's/percent/%/g' | sed 's/no-units//g' | sed 's/kilowatt-hours/kWh/g' | sed 's/cubic-meters-per-hour/m3h/g' | sed 's/watts/W/g' | sed 's/meters/m/g' | sed 's/hours/h/g' | sed 's/revolutions-per-minute/tr\/min/g' | sed 's/parts-per-million/ppm/g')
 
 
#Condition
if (( $(echo " "$valeur" > "$CRIT" " | bc -l) ));then
        echo " "$valeur" "$unity" | $unity=$valeur;$WARN;$CRIT;0;$MAX " && exit 2
elif (( $(echo " "$valeur" >= "$WARN" " | bc -l) ));then
        echo " "$valeur" "$unity" | $unity=$valeur;$WARN;$CRIT;0;$MAX " && exit 1
else
        echo " "$valeur" "$unity" | $unity=$valeur;$WARN;$CRIT;0;$MAX " && exit 0
fi

Pour faire simple, le script va chercher chercher une valeur et son unité, pour la comparer à des seuils.

Cependant, lorsque je laisse le plugin tourner sous centreon, je récupère des valeurs que je ne devrais pas récupérer...
exemple :


Je ne devrais avoir en légende que le "C°" et non pas les autres valeurs.

Donc selon vous, est ce que cela est dû à un problème dans mon script, à avoir avec centreon, ou ma commande pour récupérer les valeurs ?
Sachant que je n'ai jamais rencontré ce problème lorsque je lance mon script sous bash

[jack-ft]

Je n'ai pas tout suivi et n'ai pas trop le temps de rechercher, mais il me semble que, dans les messages précédents, il t'arrivait parfois d'avoir des valeurs erronées.
D'ailleurs, tu avais contourné ce problème en faisant, dans ce cas, une 2ème lecture des données.
1) je ne vois pas ce contournement ici
2) ce contournement ne te prémunissait pas contre le risque d'avoir 2 fois de suite des valeurs erronées

J'espère que cela pourra aider un peu...

[disedorgue]

Bonjour,

Difficile de dire, il faudrait que tu nous donnes des données généré par ton script.

Et surtout si tu as un seul type d'appel à ton script avec les arguments, car à priori, on peut l'appeler de différentes façons mais tu ne controles pas la cohérence des paramètres...

Par contre, désolé d'être désagréable, mais il y a au moins une ligne qui est extrèmement immonde ( ) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

unity=$(echo $unite | sed 's/degrees-celsius/°C/g' | sed 's/percent-relative-humidity/% humidité relative/g' | sed 's/percent/%/g' | sed 's/no-units//g' | sed 's/kilowatt-hours/kWh/g' | sed 's/cubic-meters-per-hour/m3h/g' | sed 's/watts/W/g' | sed 's/meters/m/g' | sed 's/hours/h/g' | sed 's/revolutions-per-minute/tr\/min/g' | sed 's/parts-per-million/ppm/g')

Ici, tu crées 11 process sed alors qu'un seul suffit en modifiant en quelque chose comme sed -e 's/blabla/blabla/g' -e 's/blabla/blabla/g' -e ....

[ggnore]

Le format de perfdata d'un plugin nagios c'est :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

'label'=value[UOM];[warn];[crit];[min];[max]

On dirait que tu changes 'label' à chaque fois, le fichier RRD qui héberge tes données a du mal à s'y retrouver.

Le caractère ° m'a déjà posé des problèmes au cours de l’exécution de plugins nagios et la récupération de la perfdata. Je suis plutôt enclin à n'utiliser que des caractères un minimum exotiques.

Si tes perfdatas sont inconsistantes (pas stables dans leur nommage), ton fichier RRD va se construire à partir des premières données qu'il a reçues. ça fait que par la suite il aura le mauvais format pour héberger des perfdatas correctes. Pour palier à ça, soit on pète le fichier RRD et on laisse le système le recréer fonction des données reçues. Soit on le modifie à la main avec une chance de conserver l'historique de données. Il faut exporter vers du xml, parser/modifier puis régénérer le rrd.

[musha76]

@jack-ft Oui effectivement mais je me suis rendu compte en laissant tourner le script qu'une deuxième lecture des données ne permettait pas de résoudre mon problème, donc j'ai retiré cette partie de mon script.

@disedorgue Voici pour ce qui est du détail du script :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
bash -x check.sh -A 2098212 -B 2 -C 2939 -D 30 -E 35 -F 40
+ getopts A:B:C:D:E:F:hV OPT
+ case $OPT in
+ AUTOMATE=2098212
+ getopts A:B:C:D:E:F:hV OPT
+ case $OPT in
+ OBJECT=2
+ getopts A:B:C:D:E:F:hV OPT
+ case $OPT in
+ IDENTIFIER=2939
+ getopts A:B:C:D:E:F:hV OPT
+ case $OPT in
+ WARN=30
+ getopts A:B:C:D:E:F:hV OPT
+ case $OPT in
+ CRIT=35
+ getopts A:B:C:D:E:F:hV OPT
+ case $OPT in
+ MAX=40
+ getopts A:B:C:D:E:F:hV OPT
++ /tmp/bacnet-stack-0.8.3/bin/bacrp 2098212 2 2939 85
++ sed 's/\r//'
Registering with BBMD at X.X.X.X:XXXX for 60000 seconds
+ val=21.369999
++ /tmp/bacnet-stack-0.8.3/bin/bacrp 2098212 2 2939 117
++ sed 's/\r//'
Registering with BBMD at X.X.X.X:XXXX for 60000 seconds
+ unite=degrees-celsius
++ echo 21.369999
++ cut -f1 -d.
+ valeur=21
++ echo degrees-celsius
++ sed -e 's/degrees-celsius/degrés Celsius/g' -e 's/percent-relative-humidity/% humidité relative/g' -e s/percent/%/g -e s/no-units//g -e s/kilowatt-hours/kWh/g -e s/cubic-meters-per-hour/m3h/g -e s/watts/W/g -e s/meters/m/g -e s/hours/h/g -e 's/revolutions-per-minute/tr\/min/g' -e s/parts-per-million/ppm/g
+ unity='degrés Celsius'
++ echo 21 $'degr\303\251s' Celsius
+ retour='21 degrés Celsius'
++ echo ' 21 > 35 '
++ bc -l
+ ((  0  ))
++ echo ' 21 >= 30 '
++ bc -l
+ ((  0  ))
+ echo ' 21' $'degr\303\251s' 'Celsius | degrés Celsius=21;30;35;0;40 '
 21 degrés Celsius | degrés Celsius=21;30;35;0;40
+ exit 0

@ggnore C'est pas bête... j'ai donc corrigé mon script hier pour le label et le "°C", mais ce matin je constate toujours le même problème

Voici la sortie nagios modifié :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
retour=$(echo $valeur $unity)
 
#Condition
if (( $(echo " "$valeur" > "$CRIT" " | bc -l) ));then
        echo " "$retour" | "$unity"="$valeur";"$WARN";"$CRIT";0;"$MAX" " && exit 2
elif (( $(echo " "$valeur" >= "$WARN" " | bc -l) ));then
        echo " "$retour" | "$unity"="$valeur";"$WARN";"$CRIT";0;"$MAX" " && exit 1
else
        echo " "$retour" | "$unity"="$valeur";"$WARN";"$CRIT";0;"$MAX" " && exit 0
fi

[musha76]

EDIT :
De plus, je ne comprends pas comment j'obtiens un nombre à 6 chiffres après la virgule, sachant que la commande

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

valeur=$(echo $val | cut -f1 -d\.)

me permet d'arrondir le nombre...