Script shell - datacore - problème

Version imprimable

Bonjour à toutes et à tous,

Suite à la mise en place d'un outil de supervision, je dois créer des scripts pour interroger mes OIDs de mes datacore et les intégrer à ma supervision.

Je débute en shell, voici mon code :

Code:

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
#! /bin/sh
 
STATE=2
PLUGIN_PATH='/srv/eyesofnetwork/nagios/plugins'
COMMAND_PATH='/bin'
 
   ####################################### Resultats #########################################
 
   #### valeur pour up to date du vDISK PRODUCTION ####
 
tab=(
'.32.69.52.50.50.56.68.51.66.51.69.69.54.56.68.65.65.69.69.55.57.50.68.52.55.53.55.68.52.53.57.69.70'
'.32.56.67.65.55.51.51.67.54.67.54.55.55.54.52.65.52.56.55.52.67.50.52.56.50.56.54.53.52.69.54.55.70'
'.32.68.66.69.56.65.53.54.70.66.49.70.54.57.69.52.65.68.50.69.52.67.53.52.68.67.51.51.48.70.49.57.48'
'.32.54.55.66.67.52.51.53.67.69.57.50.55.57.57.57.67.55.56.66.48.54.65.57.67.51.65.66.57.54.70.69.54'
'.32.51.56.56.66.67.68.68.51.53.69.65.57.54.50.69.53.55.69.48.65.52.69.53.68.57.54.50.53.56.48.55.49'
'.32.54.52.67.56.67.65.55.57.70.53.69.49.70.68.53.51.67.50.50.53.70.48.65.49.68.50.70.65.70.52.49.65'
'.32.52.65.70.69.68.53.53.68.48.50.53.55.49.55.54.68.57.49.49.53.70.57.51.65.51.57.69.69.65.52.57.55'
'.32.51.56.54.57.53.67.50.52.70.51.56.55.50.65.66.54.52.70.54.50.65.65.56.55.51.65.51.67.49.70.49.53'
'.32.66.53.69.52.65.57.49.70.69.54.53.51.69.51.66.65.65.66.70.48.57.70.51.53.51.65.67.51.50.57.56.69'
'.32.52.67.53.52.69.50.68.65.68.69.66.56.66.67.48.49.56.54.50.55.67.56.56.69.51.67.69.51.49.68.70.56'
   )
 
if [[ $4 = status && $6 -eq 1 || $6 -eq 2 ]]
then
for ((n=0; n < 11; n++))
do
	ar+=( "$($PLUGIN_PATH/check_snmp -H $2 -C X -o 1.3.6.1.4.1.7652.1.1.1.6"${tab[n]}" | grep to | cut -d '-' -f 2 | cut -d '"' -f 2 | cut -c -2)" )
	if [[ (("${ar[n]}" == "Up")) ]] ; then
	STATE=0
	else
	STATE=2
	fi
done
 
rslt_affiche="PRODUCTION_01 est $ar & PRODUCTION_02 est ${ar[1]} & PRODUCTION_03 est ${ar[2]} & PRODUCTION_04 est ${ar[3]} & PRODUCTION_05 est ${ar[4]} & PRODUCTION_06 est ${ar[5]} & PRODUCTION_07 est ${ar[6]} & PRODUCTION_08 est ${ar[7]} & PRODUCTION_09 est ${ar[8]} & PRODUCTION_10 est ${ar[9]}"
fi
 
if [[ ($4 == status) ]] ; then #affichage sur le nagios
   $PLUGIN_PATH/check_dummy $STATE "$rslt_affiche"
fi
   exit $STATE

Mon problème actuel

Mon script retourne toujours l'état "CRITIQUE" alors que tous mes vdisk sont UP.

Je pense que le problème vient de là (mon if dans ma boucle retourne la valeur STATE=2, elle arrive pas à lire mon if) :

Code:

1
2
3
4
5
6
 
if [[ (("${ar[n]}" == "Up")) ]] ; then
	STATE=0
	else
	STATE=2
	fi

J'ai testé plusieurs choses mais je n'arrive pas à corriger ce problème :/

Merci de votre aide.

24/02/2017, 10h58
ggnore

Salut,

Je ne sais pas si tu as essayé de faire des plugins en perl.
Pour ma part, je trouve que débuguer du perl est plus facile que bash : des types, des déclarations, des erreurs éloquentes. Tout ça aide bien à y voir clair.

Ton code m'est difficile à lire.

Code:

${tab[n]}

ton n varie vraiment dans ta boucle ?
24/02/2017, 11h08
Hujino

Citation:

Envoyé par ggnore

Salut,

Je ne sais pas si tu as essayé de faire des plugins en perl.
Pour ma part, je trouve que débuguer du perl est plus facile que bash : des types, des déclarations, des erreurs éloquentes. Tout ça aide bien à y voir clair.

Ton code m'est difficile à lire.

Code:

${tab[n]}

ton n varie vraiment dans ta boucle ?

J'ai essayé le perl et je n'y arrivais pas :/

dans ma boucle, j'ai attribué:

Citation:

ar+=

Je souhaite tester toutes mes OIDs qui sont défini dans mon "tab=" puis incrémenter à chaque fois qu'un test est fini

Mon problème :

Citation:

if [[ (${ar[$n]} = "Up") ]] ; then
STATE=0
else
STATE=2
fi

Lorsque j'ai mis :

Citation:

echo "${ar[n]}"

J'ai bien mes 10 valeurs qui sortent :D
24/02/2017, 13h47
N_BaH

Bonjour,

les parenthèses sont inutiles dans un test.

les parenthèses doubles effectue une évaluation arithmétique.
"Up" n'étant pas un nombre, il est évalué à zéro !
24/02/2017, 14h03
Hujino

Citation:

Envoyé par N_BaH

Bonjour,

les parenthèses sont inutiles dans un test.

les parenthèses doubles effectue une évaluation arithmétique.
"Up" n'étant pas un nombre, il est évalué à zéro !

Que me conseils-tu ?
24/02/2017, 14h10
N_BaH

:roll:
d'enlever les parenthèses !

Citation:

Envoyé par N_BaH

:roll:
d'enlever les parenthèses !

Oui je l'avais compris pour ceci, mais le reste? :/

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
if [[ $4 = status && $6 -eq 1 || $6 -eq 2 ]]
then
for ((n=0; n < 11; n++))
do
	ar+=( "$($PLUGIN_PATH/check_snmp -H $2 -C X -o 1.3.6.1.4.1.7652.1.1.1.6"${tab[n]}" | grep to | cut -d '-' -f 2 | cut -d '"' -f 2 | cut -c -2)" )
	if [[ ${ar[n]} == "Up" ]] ; then
	STATE=0
	else
	STATE=2
	fi
done
 
rslt_affiche="PRODUCTION_01 est $ar & PRODUCTION_02 est ${ar[1]} & PRODUCTION_03 est ${ar[2]} & PRODUCTION_04 est ${ar[3]} & PRODUCTION_05 est ${ar[4]} & PRODUCTION_06 est ${ar[5]} & PRODUCTION_07 est ${ar[6]} & PRODUCTION_08 est ${ar[7]} & PRODUCTION_09 est ${ar[8]} & PRODUCTION_10 est ${ar[9]}"
fi
 
if [[ ($4 == status) ]] ; then #affichage sur le nagios
   $PLUGIN_PATH/check_dummy $STATE "$rslt_affiche"
fi
   exit $STATE

24/02/2017, 14h44
N_BaH

tu dis qu'avec echo "{ar[n]}", tes "dix valeurs sortent", mais elles sortent quoi ?
seulement "Up" ?

en dehors de la boucle seul le dernier STATE sera affiché.

et puis, c'est pas beau grep | cut|cut|cut|cut|cut... !
à quoi ressemble le résultat du check_snmp ? on va te faire un petit awk bien propre. ;)

NB : c'est pas une très bonne pratique de poster le même sujet sur plusieurs forums.
on ne sait pas qui intervient, avec quelles directives, à quel moment des modifications...
24/02/2017, 15h16
Hujino

Yep elles me sortent "UP" et je souhaite ensuite tester si TOUT mes vdisk, si tous "UP" = OK, si un vdisk ne donne rien alors = critical.

Oui pas très esthétique mes plusieurs cut xD je suis débutant :c

Si je lance le check_snmp, la valeur me donne :

Citation:

SNMP OK - "Up to date" |

Justement je mets à jour pour éviter de se mélanger les pinceaux !
24/02/2017, 17h03
N_BaH

Citation:

Si je lance le check_snmp, la valeur me donne :

Code:

SNMP OK - "Up to date" |

pourquoi ne pas tester cette chaîne alors, simplement ?

Code:

if test "${ar[n]}" = 'SNMP OK - "Up to date" |'; then ...

quelle pourraient-être les autres chaînes retournées par check_snmp ?
24/02/2017, 17h08
disedorgue

Bonjour,

Ton premier if est affreux car susceptible de sortir une erreur de syntaxe si par exemple $6 est vide

Que t'affiche:

Code:

echo ${ar[@]}

Je ne sais pas si ça peut aider, mais voici ce que j'obtiens:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
]$ tab=(
> '.32.69.52.50.50.56.68.51.66.51.69.69.54.56.68.65.65.69.69.55.57.50.68.52.55.53.55.68.52.53.57.69.70'
> '.32.56.67.65.55.51.51.67.54.67.54.55.55.54.52.65.52.56.55.52.67.50.52.56.50.56.54.53.52.69.54.55.70'
> '.32.68.66.69.56.65.53.54.70.66.49.70.54.57.69.52.65.68.50.69.52.67.53.52.68.67.51.51.48.70.49.57.48'
> '.32.54.55.66.67.52.51.53.67.69.57.50.55.57.57.57.67.55.56.66.48.54.65.57.67.51.65.66.57.54.70.69.54'
> '.32.51.56.56.66.67.68.68.51.53.69.65.57.54.50.69.53.55.69.48.65.52.69.53.68.57.54.50.53.56.48.55.49'
> '.32.54.52.67.56.67.65.55.57.70.53.69.49.70.68.53.51.67.50.50.53.70.48.65.49.68.50.70.65.70.52.49.65'
> '.32.52.65.70.69.68.53.53.68.48.50.53.55.49.55.54.68.57.49.49.53.70.57.51.65.51.57.69.69.65.52.57.55'
> '.32.51.56.54.57.53.67.50.52.70.51.56.55.50.65.66.54.52.70.54.50.65.65.56.55.51.65.51.67.49.70.49.53'
> '.32.66.53.69.52.65.57.49.70.69.54.53.51.69.51.66.65.65.66.70.48.57.70.51.53.51.65.67.51.50.57.56.69'
> '.32.52.67.53.52.69.50.68.65.68.69.66.56.66.67.48.49.56.54.50.55.67.56.56.69.51.67.69.51.49.68.70.56'
>    )

Code:

1
2
$ echo ${#tab[@]}
10

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
$ for ((n=0; n < 11; n++)); do echo "$n: ${tab[n]}"; done
0: .32.69.52.50.50.56.68.51.66.51.69.69.54.56.68.65.65.69.69.55.57.50.68.52.55.53.55.68.52.53.57.69.70
1: .32.56.67.65.55.51.51.67.54.67.54.55.55.54.52.65.52.56.55.52.67.50.52.56.50.56.54.53.52.69.54.55.70
2: .32.68.66.69.56.65.53.54.70.66.49.70.54.57.69.52.65.68.50.69.52.67.53.52.68.67.51.51.48.70.49.57.48
3: .32.54.55.66.67.52.51.53.67.69.57.50.55.57.57.57.67.55.56.66.48.54.65.57.67.51.65.66.57.54.70.69.54
4: .32.51.56.56.66.67.68.68.51.53.69.65.57.54.50.69.53.55.69.48.65.52.69.53.68.57.54.50.53.56.48.55.49
5: .32.54.52.67.56.67.65.55.57.70.53.69.49.70.68.53.51.67.50.50.53.70.48.65.49.68.50.70.65.70.52.49.65
6: .32.52.65.70.69.68.53.53.68.48.50.53.55.49.55.54.68.57.49.49.53.70.57.51.65.51.57.69.69.65.52.57.55
7: .32.51.56.54.57.53.67.50.52.70.51.56.55.50.65.66.54.52.70.54.50.65.65.56.55.51.65.51.67.49.70.49.53
8: .32.66.53.69.52.65.57.49.70.69.54.53.51.69.51.66.65.65.66.70.48.57.70.51.53.51.65.67.51.50.57.56.69
9: .32.52.67.53.52.69.50.68.65.68.69.66.56.66.67.48.49.56.54.50.55.67.56.56.69.51.67.69.51.49.68.70.56
10:

Je propose de remplacer for ((n=0; n < 11; n++)) par for ((n=0; n < ${#tab[@]}; n++)).

25/02/2017, 09h40
jack-ft

Autre remarque:

Code:

if [[ $4 = 'status' && $6 -eq 1 || $6 -eq 2 ]]

est équivalent à:

Code:

if [[ ($4 = 'status' && $6 -eq 1) || $6 -eq 2 ]]

Ne voudrais-tu pas plutôt:

Code:

if [[ $4 = 'status' && ($6 -eq 1 || $6 -eq 2) ]]

Re-de plus, je voudrais attirer ton attention sur le statut de sortie.
Actuellement, la variable STATE est écrasée à chaque tour de boucle.
Donc elle prend pour valeur celle obtenue pour n=11, qui est hors limite.
Donc elle vaut probablement toujours 2 (qui doit correspondre à ce que tu appelles état "CRITIQUE").

Avec ma première correction proposée, la variable STATE prendrait pour valeur celle obtenue pour n=10, c'est-à-dire le statut du dernier vdisk.

Or, sémantiquement, j'imagine que tu veux que la variable STATE passe en critique, si au moins un des vdisks n'est pas UP.

Dans ce cas, il faudrait remplacer le code par:

Code:

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
#! /bin/sh
 
STATE_OK=0
STATE_CRITIQUE=2
 
STATE=${STATE_OK}
PLUGIN_PATH='/srv/eyesofnetwork/nagios/plugins'
COMMAND_PATH='/bin'
 
####################################### Resultats #########################################
 
#### valeur pour up to date du vDISK PRODUCTION ####
 
tab=(
    '.32.69.52.50.50.56.68.51.66.51.69.69.54.56.68.65.65.69.69.55.57.50.68.52.55.53.55.68.52.53.57.69.70'
    '.32.56.67.65.55.51.51.67.54.67.54.55.55.54.52.65.52.56.55.52.67.50.52.56.50.56.54.53.52.69.54.55.70'
    '.32.68.66.69.56.65.53.54.70.66.49.70.54.57.69.52.65.68.50.69.52.67.53.52.68.67.51.51.48.70.49.57.48'
    '.32.54.55.66.67.52.51.53.67.69.57.50.55.57.57.57.67.55.56.66.48.54.65.57.67.51.65.66.57.54.70.69.54'
    '.32.51.56.56.66.67.68.68.51.53.69.65.57.54.50.69.53.55.69.48.65.52.69.53.68.57.54.50.53.56.48.55.49'
    '.32.54.52.67.56.67.65.55.57.70.53.69.49.70.68.53.51.67.50.50.53.70.48.65.49.68.50.70.65.70.52.49.65'
    '.32.52.65.70.69.68.53.53.68.48.50.53.55.49.55.54.68.57.49.49.53.70.57.51.65.51.57.69.69.65.52.57.55'
    '.32.51.56.54.57.53.67.50.52.70.51.56.55.50.65.66.54.52.70.54.50.65.65.56.55.51.65.51.67.49.70.49.53'
    '.32.66.53.69.52.65.57.49.70.69.54.53.51.69.51.66.65.65.66.70.48.57.70.51.53.51.65.67.51.50.57.56.69'
    '.32.52.67.53.52.69.50.68.65.68.69.66.56.66.67.48.49.56.54.50.55.67.56.56.69.51.67.69.51.49.68.70.56'
)
 
if [[ $4 = status && ($6 -eq 1 || $6 -eq 2) ]]
then
    for ((n=0; n < ${#tab[@]}; n++))
    do
	ar+=( "$($PLUGIN_PATH/check_snmp -H $2 -C X -o 1.3.6.1.4.1.7652.1.1.1.6"${tab[n]}" | grep to | cut -d '-' -f 2 | cut -d '"' -f 2 | cut -c -2)" )
	if [[ "${ar[n]}" != "Up" ]] ; then
	    STATE=${STATE_CRITIQUE}
	fi
    done
 
    rslt_affiche="PRODUCTION_01 est $ar & PRODUCTION_02 est ${ar[1]} & PRODUCTION_03 est ${ar[2]} & PRODUCTION_04 est ${ar[3]} & PRODUCTION_05 est ${ar[4]} & PRODUCTION_06 est ${ar[5]} & PRODUCTION_07 est ${ar[6]} & PRODUCTION_08 est ${ar[7]} & PRODUCTION_09 est ${ar[8]} & PRODUCTION_10 est ${ar[9]}"
fi
 
if [[ $4 = status ]] ; then #affichage sur le nagios
    $PLUGIN_PATH/check_dummy $STATE "$rslt_affiche"
fi
exit $STATE

25/02/2017, 13h27
Hujino

Merci de ta réponse, je vais tester cela, je vous tiens au jus.

Je vous remercie de m'aider, vous êtes géniaux !

Continuez comme ça ! 8-)

Edit : Script en place, tout marche impec, là je vais me lancé dans les conditions pour faire des seuils d'alertes ^^

script pour les seuils

Bonsoir,

Actuellement je suis en train de faire un script pour faire les seuils d'alertes, j'ai pensé à ceci ?

Code:

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
#! /bin/sh
 
STATE_OK=0
STATE_WARNING=1
STATE_CRITIQUE=2
 
STATE=${STATE_OK}
PLUGIN_PATH='/srv/eyesofnetwork/nagios/plugins'
COMMAND_PATH='/bin'
 
####################################### Resultats #########################################
 
#### valeur pour up to date du vDISK PRODUCTION ####
 
tab=(
    '.32.69.52.50.50.56.68.51.66.51.69.69.54.56.68.65.65.69.69.55.57.50.68.52.55.53.55.68.52.53.57.69.70'
)
 
if [[ $4 = status && ($6 -eq 1 || $6 -eq 2) ]]
then
    for ((n=0; n < ${#tab[@]}; n++))
    do
	ar+=( "$($PLUGIN_PATH/check_snmp -H $2 -C X -o 1.3.6.1.4.1.7652.1.1.1.6"${tab[n]}" | grep to | cut -d '-' -f 2 | cut -d '"' -f 2 | cut -c -2)" )
	if [[ "${ar[n]}" != "0" ]] ; then
	    STATE=${STATE_WARNING}
        elif [[ "${ar[n]}" >= 2 ]] ; then
            STATE=${STATE_CRITIQUE}
	fi 
    done
 
    rslt_affiche="PRODUCTION_01 est $ar"
fi

Qu'en pensez-vous ?

28/02/2017, 15h20
disedorgue

J'insiste,
J'ai un fort doute sur ce que te retourne cette ligne:

Code:

ar+=( "$($PLUGIN_PATH/check_snmp -H $2 -C X -o 1.3.6.1.4.1.7652.1.1.1.6"${tab[n]}" | grep to | cut -d '-' -f 2 | cut -d '"' -f 2 | cut -c -2)" )

Pourquoi le + avant le signe égal ?
Potentiel souci avec les ouvertures et fermetures de guillemets...
28/02/2017, 15h36
Hujino

1 pièce(s) jointe(s)

Citation:

Envoyé par disedorgue

Pourquoi le + avant le signe égal ?
Potentiel souci avec les ouvertures et fermetures de guillemets...

+ pour l'incrémentation, au niveau des guillemets je n'ai aucun soucis puis si j'oublie un guillemet je vais le voir direct :p

Et mon check_snmp me permet de d'avoir d'avoir ceci à la fin :

Pièce jointe 249313
28/02/2017, 18h40
N_BaH

Citation:

Envoyé par disedorgue

Potentiel souci avec les ouvertures et fermetures de guillemets...

non, pas de problème : il y en a à l'extérieur de la substitution de commande, qui est un "contexte" différent, et tout à fait normalement à l'intérieur.

pour ma part, je persiste à dire que cet enchainement de commande est inutile. :(
et que le retour de la commande seule pourrait être testé.
01/03/2017, 10h14
Hujino

Là, je suis en train de voir pour récupérer les valeurs en temps réel des espaces disques restant mais mes OIDs que j'ai récupéré me l'indique pas xD
01/03/2017, 11h56
jack-ft
Citation:
Envoyé par Hujino

Actuellement je suis en train de faire un script pour faire les seuils d'alertes, j'ai pensé à ceci ?

Code:

1 2 3 4 5 6 7 #! /bin/sh if [[ "${ar[n]}" != "0" ]] ; then STATE=${STATE_WARNING} elif [[ "${ar[n]}" >= 2 ]] ; then STATE=${STATE_CRITIQUE} fi

Qu'en pensez-vous ?
D'abord, il y a un bug: on ne passe jamais dans le "elif"...

Ensuite, à supposer qu'on y passe, il reste un problème sémantique avec la gestion des niveaux d'erreur.

La version que j'ai proposée marche avec 2 états (OK, pas_OK), mais ne marche pas bien avec 3.

En effet, si plusieurs vdisk fonctionnent mal, le statut du dernier en dysfonctionnement écrase les précédents.

Par exemple: vdisk1 positionne STATE à CRITIQUE, puis vdisk2 positionne STATE à WARNING.

Ce n'est probablement pas ce que tu veux...
01/03/2017, 13h10
Hujino

Citation:

Envoyé par jack-ft

En effet, si plusieurs vdisk fonctionnent mal, le statut du dernier en dysfonctionnement écrase les précédents.

Par exemple: vdisk1 positionne STATE à CRITIQUE, puis vdisk2 positionne STATE à WARNING.

Ce n'est probablement pas ce que tu veux...

tout à fait... Je test des choses de mon côté aussi ^^