Discussion :

[Cabfire]

Bonjour à tous !

J'ai un problème étrange, possiblement du au fait que je ne suis pas vraiment expert en la matière

J'ai un script assez simple dont le but est d'examiner le log d'un client IPSEC et de relancer le tunnel si celui ci tombe.

Mon problème vient du fait que le script fonctionne correctement, SAUF lorsque j'essaye de l'inclure dans Crontab...

Si vous aviez une idée, je suis plus que preneur.

Voici le script, assez simple.

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
#!/bin/bash
 
TESTTUNNEL3=$(echo $(ipsec status tunnel_3) | grep -c "ESTABLISHED")
export TESTTUNNEL3
 
if [ $TESTTUNNEL3 = 1 ]
 
then
 
        echo "Tunnel_3 en fonction" >> /root/check_tunnel3.log
 
else
 
        echo "Tunnel_3 a l'arret, redemarrage > `date`"  >> /root/check_tunnel3.log
        ipsec up tunnel_3
 
fi

[al1_24]

C'est un grand classique...

Au niveau où crontab s'exécute, la variable PATH n'est pas définie.
Il est donc nécessaire que le chemin complet des commandes appelées par crontab soit précisé. De même à l'intérieur des scripts appelés par crontab.

[Cabfire]

Ah, j'avais prévenu sur le niveau ^^

Et merci, effectivement en reprenant les variables PATH de mon environnement, ça marche beaucoup mieux !

[N_BaH]

Bonjour,

j'avais prévenu sur le niveau ^^

c'était pas la peine, ça se voit tout de suite :

Code :

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

TESTTUNNEL3=$(echo $(ipsec status tunnel_3) | grep -c "ESTABLISHED")

non !

Code :

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

testTunnel3=$(ipsec status tunnel_3 | grep -c ESTABLISHED

parce que les variables tout en majuscules sont réservées aux variables d'environnement
et parce que le echo est inutile.

Code :

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

if [ $TESTTUNNEL3 = 1 ]

encore, non.

Code :

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

if ipsec status tunnel_3 | grep -q ESTABLISHED

parce que [ est une commande (un alias de test)
et que tant qu'à utiliser une commande dans un if, autant utiliser directement la commande dont on veut tester le code de retour : grep !

et finalement :

Code :

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

ipsec status tunnel_3 | grep -q ESTABLISHED && echo "Tunnel_3 en fonction" >> /root/check_tunnel3.log

c'est tout.

ce n'est rien; je ne suis pas né avec du code sous les ongles, non plus.