Discussion :

[bessonnet]

bonjour,
voici 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
#!/bin/sh
# bin/telnet
 
while [ 1 ];do
{
 
   #declaration des variables globales
   i="0";
   PORT="23";
   PING="ping -c 1 $MACHINE";
   MACHINE="@ip";
   LOGIN="login";
   PWD="pwd";
   CMD[1]="ethernet";
   CMD[2]="portState";
   CMD[3]="8";
   CMD[4]="disable";
   CMD[5]="logout";
 
   while [ $PING ] && [ i!=720 ];do
   {
      sleep 5;
      i=`expr $i + 1`;
   }
   done;
 
   #ouverture d'une session telnet sur le switch 3com
   (sleep 2; echo -e "\15"; sleep 2; echo -e "$LOGIN\15"; sleep 2; echo -e "$PWD\15"; sleep 2; echo -e "${CMD[1]}\15"; sleep 5; echo -e "${CMD[2]}\15"; sleep 5; echo -e "${CMD[3]}\15"; sleep 5; echo -e "${CMD[4]}\15"; sleep 5; echo -e "${CMD[5]}\15";) | telnet $MACHINE $PORT;
 
   while [ ! $PING ];do
   {
      sleep 5;
   }
   done;
 
}
done;
exit 0;

ce script doit me permettre de "pinguer" en continue un switch et si au bout d'une heure une on peu toujours le "pinguer" alors il faut fermer un port du switch.
mon probleme est que mon script ne marche pas.
il me met : too many arguments line 20 line 32.
je ne comprends pas pourquoi ? Auriez vous une idée ? Pouvez vous m'aider s'il vous plait ?

[bessonnet]

sur ligne 20 et ligne 32 se trouve :
ligne 20

Code :

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

1
2
3
4
5
6
while [ $PING ] && [ i!=720 ];do
   {
      sleep 5;
      i=`expr $i + 1`;
   }
   done;

ligne 32

Code :

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

1
2
3
4
5
while [ ! $PING ];do
   {
      sleep 5;
   }
   done;

voila je prends toute les remarques et commentaires constructif qui me permettront d'avancer.
je remercie tous ceux qui ont pris le temps de lire ce topic.
Cordialement

[Arnaud F.]

Déjà dans dans ta seconde boucle while, sache que la comparaison ne se fait pas avec != mais -ne (pour Not Equal donc ta ligne deviendra

Code :

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

while [ $PING  && i -ne 720 ];do

Et il ne faut pas mettre d'accolades non plus dans la boucle while

[edit] il faut faire l'opérateur logique dans les crochets aussi (j'ai modifié le code)

[edit2] pas de ; après le done

[bessonnet]

et pour :

Code :

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

while [ ! $PING ];do

j'aurais donc :

Code :

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

while [ -ne $PING ];do

si j'ai bien compris ?
je vai essayer
merci

[bessonnet]

j'ai donc effectué les modifications,
pour la ligne 32, au premier tour l'erreur à disparue, au tour d'après elle est revenue, la meme : "too many arguments".
pour la ligne 20 elle s'est changée en : [:missing`]'
voila je m'y connais pas encore assez pour tout comprendre.
si vous pouviez continiuer à me mettre sur la bonne voie je vous en serais reconnaissante.
merci

[gorgonite]

pourquoi pas plutôt cela pour le test de ping ???

Code :

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

 [$PING -ne 0]

[Arnaud F.]

pourquoi pas plutôt cela pour le test de ping ???

Code :

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

 [$PING -ne 0]

+1

j'avais meme pas fait attention à ça ...

[Arnaud F.]

et pour :

Code :

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

while [ ! $PING ];do

j'aurais donc :

Code :

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

while [ -ne $PING ];do

si j'ai bien compris ?
je vai essayer
merci

Non car quand tu utilise -ne il faut comparer par rapport à une valeur or si tu fais ça, ce n'est pas le cas

Suis ce qu'a écris gorgonite

P.S: Pense à le modifier dans tes 2 boucles

[bessonnet]

merci,
j'ai changé dans les deux boucles, mais les erreurs persistes !
peu etre y à t'il des erreurs ailleurs que sur ces deux lignes là ? parce que là je ne vois vraiment pas ...
quand je fais : "sh -n ping.sh" il ne m'affiche aucune erreur, c'est quand je le lance (sh ping.sh) qu'il commence son execution et en meme temps il m'affiche ces deux erreurs.
au finale il tourne en rond avec les deux erreurs.

auriez vous d'autres idées ?

[Arnaud F.]

Oui, une seule, post nous ton code mis à jour maintenant, qu'on voit réellement ce que tu à modifié

[bessonnet]

voici le code

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/sh
# bin/telnet
 
while [ 1 ];do
{
 
   #declaration des variables globales
   i="0";
   PORT="23";
   PING="ping -c 1 @ip";
   MACHINE="@ip";
   LOGIN="login";
   PWD="pwd";
   CMD[1]="ethernet";
   CMD[2]="portState";
   CMD[3]="8";
   CMD[4]="disable";
   CMD[5]="logout";
 
   while [ [ $PING ] && [ i -ne 720 ] ]; do
   {
      sleep 5;
      i=`expr $i + 1`;
   }
   done
 
   #ouverture d'une session telnet sur le switch 3com
   (sleep 2; echo -e "\15"; sleep 2; echo -e "$LOGIN\15"; sleep 2; echo -e 
 
"$PWD\15"; sleep 2; echo -e "${CMD[1]}\15"; sleep 5; echo -e "${CMD[2]}\15"; sleep 
 
5; echo -e "${CMD[3]}\15"; sleep 5; echo -e "${CMD[4]}\15"; sleep 5; echo -e 
 
"${CMD[5]}\15";) | telnet $MACHINE $PORT;
 
   while [ $PING -ne 0 ]; do
   {
      sleep 5;
   }
   done
 
}
done
 
exit 0;

[Arnaud F.]

Enlève les accolades dans les boucles while
Et tu veux tester tant que $PING égal à quoi?

Au passage tu n'a pas besoin de mettre tes sous-expression entre crochet:

Code :

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

while [  $PING  &&  i -ne 720  ];

Voilà essai déjà ça

[Arnaud F.]

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
#!/bin/sh
# bin/telnet

while [ 1 ]; do
   #declaration des variables globales
   i="0";
   PORT="23";
   PING="ping -c 1 @ip";
   MACHINE="@ip";
   LOGIN="login";
   PWD="pwd";
   CMD[1]="ethernet";
   CMD[2]="portState";
   CMD[3]="8";
   CMD[4]="disable";
   CMD[5]="logout";

   # Tu veux tester le PING pour quelle valeur?
   while [  $PING -eq TaValeur  &&  i -ne 720  ]; do
      sleep 5;
      i=`expr $i + 1`;
   done
  
   #ouverture d'une session telnet sur le switch 3com
   (sleep 2; echo -e "\15"; sleep 2; echo -e "$LOGIN\15"; sleep 2; echo -e 

"$PWD\15"; sleep 2; echo -e "${CMD[1]}\15"; sleep 5; echo -e "${CMD[2]}\15"; sleep 

5; echo -e "${CMD[3]}\15"; sleep 5; echo -e "${CMD[4]}\15"; sleep 5; echo -e 

"${CMD[5]}\15";) | telnet $MACHINE $PORT;

   while [ $PING -ne 0 ]; do
      sleep 5;
   done
done

exit 0;

Voilà, un code qui me semble un peu plus juste, teste déjà ça et dis nous si tu as encore des erreurs

[bessonnet]

en enlevant les crochet j'ai essayer dejà, ça ne change rien et comme ej trouve ça plus clair avec je les laisse.
ce que je veux c'est "pinguer" le switch et tant qu'il repond attendre 1h et au bout de cette heure fermer le port correspondant du switch.
le port du switch je le connais.

et j'ai vu que j'avais oublié un $ devant mon i :

Code :

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

while [  $PING -eq TaValeur  &&  $i -ne 720  ]; do

donc je ne sais pas si ta ligne de code est correct, est ce que pour verifier si ça ping je dois donner une valeur à $ping ?
si oui quel genre ?

[Arnaud F.]

Dl pour le $ du i mais c'est une erreur de copié-coller

En ce qui concerne la valeur du ping à testé je ne peux pas t'aider car ça sort de mes compétences (assez limité comme tu peux le constater )

Il te renvoi quelles erreurs maintenant? toujours les meme?

[edit] Je viens de voir qqchose

Code :

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

 PING="ping -c 1 @ip";

Tu veux que ton shell script execute la commande ping -c 1 @ip à partir du shell non?
Si c'est le cas, tu dois mettre des backquotes pour executer la commande:

Code :

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

 PING=`ping -c 1 @ip`;

[bessonnet]

merci pour cette dernière info je ne savais pas cela !
je vai mettre des backquotes tout de suite !
merci