Problème boucle for

**tetzispa** · 20/06/2014, 10h10

Bonjour,

je suis sur un script afin de détecter sur des ports sur des équipements CISCO le spanning tree et leurs coûts.

Mon script fonctionne bien pour un seul host. Quand j’intègre à mon script une boucle afin de lire ligne par ligne un fichier d'hôtes, mon script ne lit que la 1ere ligne et s'arrête sans que je sache pourquoi

si quelqu'un pouvait m'expliquer le pourquoi du comment et me débuguer ! merci à vous par avance !

(inutile de vous dire que j'ai vérifié mon fichier texte, il est sous un bon format et j'ai bien une entrée par lignes

)

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
#!/bin/bash
 
#########################################################################################
#                                                                                       #
#                                                                                       #
#   Requirements:               snmpwalk, snmpget (available in NetSNMP package)        #
#   Last modification:          03/06/2014                                              #
#   Purpose of this plugin:     detection de cout STP sur les ports			    		#
#                                                                                       #
#   Arguments:                                                                          #
#                ./check-multi-stp-cisco.sh community snmp_version 	    				#
#                                            $1          $2                         	#
#########################################################################################
 
# fichier de correspondance
 
file_stp='/usr/local/nagios/type_stp.txt'
 
# fichier cisco
 
list_cisco=`cat /usr/local/nagios/cisco.txt `
 
# OID STP cisco
 
OID_STP_COST='1.3.6.1.2.1.17.2.15.1.5'
OID_PORT_STP='1.3.6.1.2.1.17.2.15.1.3'
OID_DESC_PORT='1.3.6.1.2.1.17.1.4.1.2'
OID_STP_TYPE='1.3.6.1.4.1.9.9.82.1.6.1'
 
for a in $list_cisco
 
do
 
# on vérifie l'état du STP sur le switch
type_stp=$(snmpwalk -c $1 -v$2 $a $OID_STP_TYPE | awk {'print $4'})
type_stp_label=$(cat $file_stp | grep -w $type_stp | cut -d ":" -f2)
 
 
# on regarde le nombre de ports qu'il y a de configuré en STP
ports_stp=$(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l)
 
	if [ $ports_stp = 0 ]
		then 
		echo "pas de ports en STP"
		exit 0
 
	elif test $(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l ) -gt 1
		then
			for (( i=$( snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l) ; i>0 ; i-=1 ))
				do		
					ports_stp_isole=$(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | tail -n $i | head -n 1)
					cout_ports_stp_isole=$(snmpwalk -c $1 -v$2 $a $OID_STP_COST.$ports_stp_isole | awk {'print $4'} | tail -n $i | head -n 1)
					PORTS_isole=$(snmpwalk -c $1 -v$2 $a $OID_DESC_PORT.$ports_stp_isole | awk {'print $4'} | tail -n $i | head -n 1)
					ports_desc_stp_isole=$(snmpwalk -c $1 -v$2 $a IF-MIB::ifDescr.$PORTS_isole | awk {'print $4'} | tail -n $i | head -n 1)
					stp=$"[le cout de $ports_desc_stp_isole est de $cout_ports_stp_isole]"
					total_stp="$total_stp $stp"
				done
 
					echo $a est en $type_stp_label
					echo $total_stp
					exit 1
 
			echo erreur de script
			exit 2
	fi 
 
done

**tetzispa** · 20/06/2014, 10h58

ah j'ai trouvé mon erreur, problème avec mes exit 0/1/2 :p

je les ai retiré, ça fonctionne maintenant

**ok.Idriss** · 21/06/2014, 12h13

Bonjour.

Bon ton problème est passé en résolu, mais je vais quand même me permettre quelque remarque sur ton script

Code :

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

1
2
3
4
5
list_cisco=`cat /usr/local/nagios/cisco.txt `
###
for a in $list_cisco
do
done

Cette façon de lire un fichier est totalement à proscrire ! Je te recommande de lire ces quelques liens :

Quelques bonnes pratiques dans l'écriture de scripts en Bash
[FAQ] Pourquoi faut-il éviter d'utiliser la boucle for sur une sortie de commande ?
[FAQ] Comment lire/parcourir un fichier ?

En gros ça traite de comment lire un fichier correctement en Bash (ne jamais utiliser cat pour lire un fichier car c'est totalement inutile et pas prévu pour ça et de manière générale éviter le for pour lire une sortie de commande).

---------------------------------

Code :

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

1
2
# on vérifie l'état du STP sur le switch
type_stp=$(snmpwalk -c $1 -v$2 $a $OID_STP_TYPE | awk {'print $4'})

awk ici c'est un peu comme sortir l'artillerie lourde pour pas grand chose, surtout que Bash sait très bien gérer ce genre de cas :

Code :

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

1
2
3
# on vérifie l'état du STP sur le switch
read -ra outSnmpwalk < <(snmpwalk -c $1 -v$2 $a $OID_STP_TYPE)
type_stp="${outSnmpwalk[3]}"

---------------------------------

Code :

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

type_stp_label=$(cat $file_stp | grep -w $type_stp | cut -d ":" -f2)

Le cat est à nouveau totalement superflus ! grep sait très bien lire un fichier en entrée :

Code :

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

type_stp_label=$(grep -w "$type_stp" "$file_stp" | cut -d ":" -f2)

Bon là il faut savoir que Bash aurrait pu savoir remplacer le cut à l'aide d'un changement d'IFS, mais bon cut est très bien adapté pour les petits découpages de ce type.
Si on voulais éviter la multiplication de processus, un awk aurait pu convenir également :

Code :

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

type_stp_label=$(awk -F ":" -vword="$type_stp" '($0 ~ word){print $2}' "$file_stp")

Si l'option "-w" de grep était indispensable, on peux compenser par :

Code :

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

type_stp_label=$(awk -F ":" -vword="$type_stp" '($0 ~ "^(.* )?"word"( .*)?$"){print $2}' "$file_stp")

Bien sûr ici je chipote un peu, le grep + cut c'est ce que j'aurais probablement utilisé, mais c'est pour sensibiliser au fait qu'il faille toujours réfléchir à éviter la multiplication des processus inutilement (cat|grep|cut par exemple).

---------------------------------

Code :

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

1
2
# on regarde le nombre de ports qu'il y a de configuré en STP
ports_stp=$(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l)

Dès lors qu'il y a plusieurs délimiteurs, awk devient plus adapté que cut :

Code :

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

awk -F "[.=]" '{print $1}'

---------------------------------

Code :

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

1
2
3
4
5
if [ $ports_stp = 0 ]
		then 
		#...
 
	elif test $(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l ) -gt 1

Bon ici, je m'abstiendrais de commentaire sur l'identation (bien que ce soit quand même quelque chose de très important y compris dans l'écriture de script shell)

Quelques remarques toutefois sur cet extrait :

1) Il faut choisir les bons opérateurs dans son test ;
2) Il faut protéger ses opérandes.
3) Je trouve curieux le mélange de syntaxe (test et [) et d'opérateurs

---------------------------------

Code :

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

1
2
3
4
5
6
7
8
9
elif test $(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l ) -gt 1
#...
for (( i=$( snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l) ; i>0 ; i-=1 ))
do	
     ports_stp_isole=$(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | tail -n $i | head -n 1)
     cout_ports_stp_isole=$(snmpwalk -c $1 -v$2 $a $OID_STP_COST.$ports_stp_isole | awk {'print $4'} | tail -n $i | head -n 1)
     PORTS_isole=$(snmpwalk -c $1 -v$2 $a $OID_DESC_PORT.$ports_stp_isole | awk {'print $4'} | tail -n $i | head -n 1)
     ports_desc_stp_isole=$(snmpwalk -c $1 -v$2 $a IF-MIB::ifDescr.$PORTS_isole | awk {'print $4'} | tail -n $i | head -n 1)
# ...

Il semble qu'il y ai beaucoup de redondance sur les appels et le découpage des retour de snmpwalk, malheureusement je connais pas cette commande.
Peut être qu'en nous fournissant un flux entrant et ce que tu souhaites en retirer, nous pourrions t'indiquer des solutions davantage optimales...

Cordialement,
Idriss

**Winnt** · 21/06/2014, 19h40

Chipoteur

Mais toujours intéressantes comme remarques.
Je tâcherai de les avoir à l'esprit lorsque je ferai un script.

**tetzispa** · 21/06/2014, 20h01

hello !

merci pour ta réponse

bon apparemment, c'est pas aussi bien et optimisé que ça

:p mais ça à le mérite de fonctionner ^^

en gros, c'est un script pour nagios pour faire de la supervision.

le SNMP sert à ça, le fait d'installer net-snmp sur un linux te permet de passer des commandes et d'avoir des retours sur des équipements, car chaque équipement à une MIB qui sommeille en lui et donc le fait d'interroger en Read Only (RO) une certain OID te renverra une information que tu pourras traiter, ce qui va donc plus vite que de te connecter en ssh, de passer une commande et de récupérer le retour de ta commande sur l'équipement

pour te montrer un exemple, voilà ce que fait du SNMPWALK sur un équipement cisco (c'est lié par rapport à mon 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
 
file='/usr/local/nagios/stp.txt'
#1:disabled
#2:blocking
#3:listening
#4:learning
#5:forwarding
#6:broken
 
# OID PORT : 1.3.6.1.2.1.17.1.4.1.2
# on récupère donc le 52 et 108 avec un : cut -d"." -f12 | cut -d'=' -f1
 
root [/usr/local/nagios/libexec] > snmpwalk -c adelweb94public -v1 occis4 1.3.6.1.2.1.17.1.4.1.2
iso.3.6.1.2.1.17.1.4.1.2.52 = INTEGER: 10152
iso.3.6.1.2.1.17.1.4.1.2.108 = INTEGER: 10652
 
 
 
# OID STATUS : 1.3.6.1.2.1.17.2.15.1.3
# le 52 et 108 qu'on a recup avant on le met à la fin de l'OID et on a l'état et on le compare avec le fichier d'état 
 
root [/usr/local/nagios/libexec] > snmpwalk -c adelweb94public -v1 occis4 1.3.6.1.2.1.17.2.15.1.3.52
iso.3.6.1.2.1.17.2.15.1.3.52 = INTEGER: 2
¨
 
# OID Description port : 1.3.6.1.2.1.2.2.1.2
# on trouve la description du port avec l'OID du port (avant) sauf que là on recupère la valeur après l'INTEGER 
# cad : 10152 et 10652
 
root [/usr/local/nagios/libexec] > snmpwalk -c adelweb94public -v1 occis4 1.3.6.1.2.1.2.2.1.2.10152
iso.3.6.1.2.1.2.2.1.2.10152 = STRING: "GigabitEthernet1/0/52"
 
# l'affichage voulu sera du genre :
 
GigabitEthernet1/0/52 - blocking
GigabitEthernet2/0/52 - forwarding
 
# donc pour les états 2 et 5 = OK
# et le truc infâme : si un port est en 2 ou 5, si son état change = afficher un CRITQUE avec la cause

si avec tout ce que tu m'as dis, tu me trouves un moyen d'optimiser, de fluidifier et de sécuriser tout mon code, je suis preneur !

merci encore à toi pour ton retour et tes conseils forts utiles

**tetzispa** · 21/06/2014, 20h09

je pense qu'avec des "case" par exemple ça serait mieux que de lire un fichier qui comporte mes erreurs etc. (tu me diras si je me trompe

)

mais je sais pas faire :p

**ok.Idriss** · 22/06/2014, 10h26

Re-bonjour.

Bon alors déjà comme pistes d'optimisations, tu en as déjà pas mal dans mon précédent message, notamment :

1) Remplacer :

Code :

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

1
2
3
4
5
6
list_cisco=`cat /usr/local/nagios/cisco.txt `
#...
for a in $list_cisco
do
    # ...
done

Par quelque chose comme

Code :

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

1
2
3
4
fileCisco="/usr/local/nagios/cisco.txt"
while read -r a; do
    # ...
done < "$fileCisco"

2) De manière générale, éviter la multiplication des processus et des programmes externes, virer tout les cat superflus comme expliqué dans mon précédent message, etc.

Déjà si tu fait tout ça, ça sera beaucoup mieux et ça ne te coûtera pas grand chose normalement. Après quand je disais qu'il y avait beaucoup de redondance je parlais des nombreux appels à snmpwalk qui se répètent. Je ne sais pas quel est le coût de cette commande toutefois... mais il est surement possible de stocker un résultat et le parser plutôt que de rappeler cette commande X fois, d'autant qu'elle est plusieurs fois appelée avec exactement les mêmes paramètres :

Code :

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

1
2
3
elif test $(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l ) -gt 1
#...
for (( i=$( snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l) ; i>0 ; i-=1 ))

Pourquoi ne pas stocker $(snmpwalk -c $1 -v$2 $a $OID_PORT_STP | cut -d"." -f12 | cut -d'=' -f1 | wc -l ) dans une variable afin d'éviter de recalculer la même chose deux fois d'affilé ?

D'ailleurs les cut ne servent pas à grand chose ici, ils ne filtreront pas les lignes qui contiennent "." et "=", donc à priori :

Code :

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

snmpwalk -c $1 -v$2 $a $OID_PORT_STP|cut -d"." -f12 | cut -d'=' -f1 | wc -l

Renverra toujours la même chose que :

Code :

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

snmpwalk -c $1 -v$2 $a $OID_PORT_STP|wc -l

Sans oublier ma remarque de mon précédent message sur les séparateurs multiples

je pense qu'avec des "case" par exemple ça serait mieux que de lire un fichier qui comporte mes erreurs etc. (tu me diras si je me trompe )

mais je sais pas faire :p

Bah je ne sais pas trop ce que tu entend par "un fichier qui comporte mes erreurs" ni ce que tu souhaite faire avec case donc je peux pas trop te dire si c'est faux ou pas

.

Cordialement,
Idriss

**ok.Idriss** · 22/06/2014, 10h29

Pour la suite, je préfère poster dans un nouveau message :

Code :

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

1
2
3
4
5
6
# OID PORT : 1.3.6.1.2.1.17.1.4.1.2
# on récupère donc le 52 et 108 avec un : cut -d"." -f12 | cut -d'=' -f1
 
root [/usr/local/nagios/libexec] > snmpwalk -c adelweb94public -v1 occis4 1.3.6.1.2.1.17.1.4.1.2
iso.3.6.1.2.1.17.1.4.1.2.52 = INTEGER: 10152
iso.3.6.1.2.1.17.1.4.1.2.108 = INTEGER: 10652

=>

Code :

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

1
2
3
4
5
6
$ cat test.txt 
iso.3.6.1.2.1.17.1.4.1.2.52 = INTEGER: 10152
iso.3.6.1.2.1.17.1.4.1.2.108 = INTEGER: 10652
$ awk -F "[.=]" '{print $12}' test.txt
52
108

Bien sûr dans ton cas, ça sera plutôt :

Code :

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

$ snmpwalk -c adelweb94public -v1 occis4 1.3.6.1.2.1.17.1.4.1.2|awk -F "[.=]" '{print $12}'

-----------------------------------------------

Code :

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

1
2
3
# OID Description port : 1.3.6.1.2.1.2.2.1.2
# on trouve la description du port avec l'OID du port (avant) sauf que là on recupère la valeur après l'INTEGER 
# cad : 10152 et 10652

=>

Code :

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

1
2
3
4
5
6
$ cat test.txt 
iso.3.6.1.2.1.17.1.4.1.2.52 = INTEGER: 10152
iso.3.6.1.2.1.17.1.4.1.2.108 = INTEGER: 10652
$ awk -F "[.=:]" '{gsub(/(^[ \t]*|[ \t]*$)/, "", $14); print $14}' test.txt # pareil à piper sur le retour de ta commande
10152
10652

Cordialement,
Idriss

**disedorgue** · 22/06/2014, 11h35

Bonjour,

Envoyé par ok.Idriss

Code :

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

1
2
3
4
5
6
$ cat test.txt 
iso.3.6.1.2.1.17.1.4.1.2.52 = INTEGER: 10152
iso.3.6.1.2.1.17.1.4.1.2.108 = INTEGER: 10652
$ awk -F "[.=:]" '{gsub(/(^[ \t]*|[ \t]*$)/, "", $14); print $14}' test.txt # pareil à piper sur le retour de ta commande
10152
10652

Ici, on sait que c'est un entier, on pourrait donc peut-être simplifier

:

Code :

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

awk -F "[.=:]" '{printf "%u\n",$14}' test.txt

@tetzispa: Pour ce qui est d'optimiser le script, il y a du boulot, mais déjà tu pourrais par exemple mettre dans une variable le résultat de commande que tu utilises plusieurs fois avec les mêmes paramètres, au lieu de rappeler n fois la commande.

**ok.Idriss** · 22/06/2014, 11h39

Envoyé par disedorgue

Bonjour,

Ici, on sait que c'est un entier, on pourrait donc peut-être simplifier

:

Code :

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

awk -F "[.=:]" '{printf "%u\n",$14}' test.txt

Ben j'en étais pas sûr justement (il y a des cas ou visiblement ça renvoie "STRING" suivi d'une chaîne de caractère

).

Envoyé par disedorgue

@tetzispa: Pour ce qui est d'optimiser le script, il y a du boulot, mais déjà tu pourrais par exemple mettre dans une variable le résultat de commande que tu utilises plusieurs fois avec les mêmes paramètres, au lieu de rappeler n fois la commande.

Tout à fait d'accord, c'est de ça que je parlais en parlant de "redondance" !

**disedorgue** · 22/06/2014, 12h06

Envoyé par ok.Idriss

Ben j'en étais pas sûr justement (il y a des cas ou visiblement ça renvoie "STRING" suivi d'une chaîne de caractère

).

Je ne connais pas trop le protocole dérrière, c'est aussi pour ça que je parle au conditionnel

**tetzispa** · 22/06/2014, 12h18

lol on poste un truc, on se réveille le lendemain avec des milliers de lignes de codes et d'explications

je vais tester et tenter de mettre ça en œuvre demain au taf, ça va pas être joyeux

**tetzispa** · 23/06/2014, 08h32

hello !

bon mes galères commencent ...

voici ce que je tente de faire pour commencer (et qui donc ne fonctionne pas

) je loupe un truc évident (pour vous) quelque part mais que je ne vois pas :

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
# fichier cisco
 
fileCisco='/usr/local/nagios/cisco2.txt'
 
# OID STP cisco
 
OID_STP_COST='1.3.6.1.2.1.17.2.15.1.5'
OID_PORT_STP='1.3.6.1.2.1.17.2.15.1.3'
OID_DESC_PORT='1.3.6.1.2.1.17.1.4.1.2'
OID_STP_TYPE='1.3.6.1.4.1.9.9.82.1.6.1'
 
# variables pour SNMP
 
community='adelweb94public'
version='1'
 
# commande
 
cmd=$(snmpwalk -c $community -v$version $a)
 
while read a; do
 
type_stp=$($cmd $OID_STP_TYPE | awk {'print $4'})
type_stp_label=$(cat $file_stp | grep -w $type_stp | cut -d ":" -f2)
 
echo $a est en $type_stp_label
 
done < "$fileCisco"

et le programme n'arrive pas à tourner. Il m'affiche bien les noms des switches (donc ça il arrive à lire dedans) mais ce qui est étonnant c'est que sur ma commande de SNMP il me dit qu'il ne trouve pas l'hôte ?_?

Code :

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

1
2
root [/usr/local/nagios/libexec] > ./check-multi-stp-cisco.sh
No hostname specified.

Code :

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

1
2
3
4
./check-multi-stp-cisco.sh: ligne39: 1.3.6.1.4.1.9.9.82.1.6.1 : commande introuvable
Usage: grep [OPTION]... MOTIF [FICHIER]...
Try 'grep --help' for more information.
occis4 est en

je dois donc merder quelque chose dans mes déclarations de variables et dans leurs appels :/ mais je n'arrive pas à voir où

**ok.Idriss** · 23/06/2014, 09h18

Bonjour.

Code :

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

type_stp=$($cmd $OID_STP_TYPE | awk {'print $4'})

$cmd n'est pas une commande. Il s'agit du résultat d'une commande :

Code :

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

cmd=$(snmpwalk -c $community -v$version $a)

Il faudrait soit passer par un eval (et stocker la commande sous forme de chaine de caractère et non dans un sous-shell) pour être interprété en tant que commande (mais c'est dangereux), soit passer par une fonction Bash pour éviter de répeter la commande et ses paramètres si c'est ce que tu souhaite faire. La seconde solution est préférable

Encore une fois, awk ici c'est pas utile, autant utiliser une syntaxe Bash à base de builtin (voir mon premier post)

-----------------------------------

Code :

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

type_stp_label=$(cat $file_stp | grep -w $type_stp | cut -d ":" -f2)

Le cat est superflus, encore une fois :

Code :

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

type_stp_label=$(grep -w "$type_stp" "$file_stp" | cut -d ":" -f2)

-----------------------------------

Sinon il y a surement un autre grep qui plante, peux-tu nous fournir la ligne 39 de ton script ?

Idriss

**tetzispa** · 23/06/2014, 09h23

je travaille par bloc là, en gros je teste le while read, après je m'attarde au reste (d'où le fait que tu vois encore l'awk, les cat etc

)

pour :

soit passer par une fonction Bash pour éviter de répeter la commande et ses paramètres si c'est ce que tu souhaite faire

c'est à dire ?

ma ligne 39 :

Code :

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

1
2
type_stp=$($cmd $OID_STP_TYPE | awk {'print $4'})
type_stp_label=$(grep -w "$type_stp" "$file_stp" | cut -d ":" -f2)

le grep déconne là dedans

**tetzispa** · 23/06/2014, 10h19

bon j'avais pas les yeux en face des trous pour le coup de la fonction

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
# fichier de correspondance
 
file_stp='/usr/local/nagios/type_stp.txt'
folder='/usr/local/nagios/STP'
 
# fichier cisco
 
fileCisco='/usr/local/nagios/cisco2.txt'
 
# OID STP cisco
 
OID_STP_COST='1.3.6.1.2.1.17.2.15.1.5'
OID_PORT_STP='1.3.6.1.2.1.17.2.15.1.3'
OID_DESC_PORT='1.3.6.1.2.1.17.1.4.1.2'
OID_STP_TYPE='1.3.6.1.4.1.9.9.82.1.6.1'
 
# variables pour SNMP
 
community='adelweb94public'
version='-v1'
 
# fonction commande 
 
type_stp () {
 
snmpwalk -c $community $version $a $OID_STP_TYPE | awk {'print $4'}
 
}
 
while read a; do
 
type_stp_label=$(grep -w type_stp "$file_stp" | cut -d ":" -f2)
type_stp
 
echo $a est en $type_stp_label
 
done < "$fileCisco"

(pas d'indentation, toujours un awk, c'est normal :p)

la commande marche à moitié, ça m'affiche bien un "5" mais impossible d'avoir la correspondance avec mon fichier sans que je comprenne trop pourquoi :

Code :

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

1
2
3
root [/usr/local/nagios/libexec] > ./check-multi-stp-cisco.sh
5
occis4 est en

il devrait m'afficher "occis4 est en rapidPvstPlus"

**Flodelarab** · 23/06/2014, 10h43

Bonjour,

Et quand tu trackes l'erreur, jusqu'où cela est-il mauvais?

Ce que je veux dire c'est que ton cut peut ne pas être bon si ta ligne ne contient pas ':'
Ton grep peut être mauvais.
etc...

Donc, jusqu'où remonte l'erreur?

**tetzispa** · 23/06/2014, 10h45

la ligne contient toujours un ":"

mais là je suis passé à l'étape suivante lol :p

ma fonction m'affiche bien le code que je cherche (en l'occurrence 5 ici qui correspond à rapidPvstPlus dans mon fichier) et quand j'appelle le retour de ma fonction dans une commande, ça m'affiche du vide comme j'ai pu mettre juste avant

donc là je sèche un peu (on va me dire qu'on peut pas rappeler un retour de fonction dans une commande

)

**tetzispa** · 23/06/2014, 13h19

bon j'avance et j'ai compris mon erreur !

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
# variables pour SNMP
 
community='adelweb94public'
version='-v1'
 
# fonction commande 
 
type_stp () {
 
cmd=$(snmpwalk -c $community $version $a $OID_STP_TYPE | awk {'print $4'})
 
}
 
while read a; do
 
type_stp
type_stp_label=$(grep -w "2" "$file_stp" | cut -d ":" -f2 )
echo $a est en $type_stp_label
 
done < "$fileCisco"

et ça fonctionne parfaitement, je continue à modifier la suite

**tetzispa** · 23/06/2014, 14h57

bon je pense que je suis pas mal maintenant

le script fonctionne, quels seraient des axes d'améliorations/optimisations (j'ai du encore louper 2/3 trucs

) que vous me conseillerez ?

merci en tout cas à vos remarques constructives

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
#!/bin/bash
 
#########################################################################################
#                                                                                       #
#                                                                                       #
#   Requirements:               snmpwalk, snmpget (available in NetSNMP package)        #
#   Last modification:          03/06/2014                                              #
#   Purpose of this plugin:     detection de cout STP sur les ports			    		#
#                                                                                       #
#   Arguments:                                                                          #
#                ./check-multi-stp-cisco.sh community snmp_version 	    				#
#                                            $1          $2                         	#
#########################################################################################
 
# fichier de correspondance
 
file_stp='/usr/local/nagios/type_stp.txt'
folder='/usr/local/nagios/STP'
 
# fichier cisco
 
fileCisco='/usr/local/nagios/cisco.txt'
 
# OID STP cisco
 
OID_STP_COST='1.3.6.1.2.1.17.2.15.1.5'
OID_PORT_STP='1.3.6.1.2.1.17.2.15.1.3'
OID_DESC_PORT='1.3.6.1.2.1.17.1.4.1.2'
OID_STP_TYPE='1.3.6.1.4.1.9.9.82.1.6.1'
 
# variables pour SNMP
 
community='adelweb94public'
version='-v1'
 
# fonction de base STP
 
fn_stp () {
 
cmd=$(snmpwalk -c $community $version $a $OID_PORT_STP | awk -F "[.=]" '{print $12}' | wc -l)
 
}
 
fn_stp2 () {
 
cmd1=$(snmpwalk -c $community $version $a $OID_PORT_STP | awk -F "[.=]" '{print $12}' | tail -n $i | head -n 1)
 
}
 
# fonction pour afficher le mode de fonctionnement du STP sur l'equipement cisco
 
type_stp () {
 
cmd2=$(snmpwalk -c $community $version $a $OID_STP_TYPE | awk {'print $4'})
 
}
 
# on fait une boucle pour lire chaque ligne du fichier de correspondance des équipements cisco
 
while read a; do
 
# on execute les différentes fonctions
 
fn_stp
type_stp
 
# on va verifier le label du STP sur l'equipement cisco
 
type_stp_label=$(awk -F ":" -vword="$cmd2" '($0 ~ word){print $2}' "$file_stp")
 
# on regarde si $cmd est égal ou non à 0, si c'est égal à 0, alors pas de ports en STP, sinon on va effectuer une verification
	if [[ $cmd = "0" ]]
		then 
		echo "$a n'a pas de ports en STP" > $folder/$a
 
	elif [[ $cmd != "0" ]]
 
# on initialise la variable total_stp afin qu'elle se vide à chaque passage pour chaque equipement	
 
	total_stp=' '
 
		then			
			for (( i=$cmd ; i>0 ; i-=1 ))
 
				do	
 
# on execute la dernière fonction
 
fn_stp2	
 
# on se met à chercher quels sont les ports qui font du STP, et on va chercher leurs couts		
 
					cout_ports_stp_isole=$(snmpwalk -c $community $version $a $OID_STP_COST.$cmd1 | awk {'print $4'} | tail -n $i | head -n 1)
					PORTS_isole=$(snmpwalk -c $community $version $a $OID_DESC_PORT.$cmd1 | awk {'print $4'} | tail -n $i | head -n 1)
					ports_desc_stp_isole=$(snmpwalk -c $community $version $a IF-MIB::ifDescr.$PORTS_isole | awk {'print $4'} | tail -n $i | head -n 1)
					stp=$"[le cout de $ports_desc_stp_isole est de $cout_ports_stp_isole]"
					total_stp="$total_stp $stp"
 
				done
 
						echo $a est en $type_stp_label > $folder/$a
						echo $total_stp >> $folder/$a		
 
	fi
 
done < "$fileCisco"

par contre, pour l'indentation, je ne vois pas trop comment mieux faire, j'ai pas d'indentation automatique sous notepad++ pour le shell je crois (outre la touche tab)

Problème boucle for

Shell et commandes GNU

Discussions similaires

Partager

Partager