Discussion :

[zorooo]

Salutation !

Je rencontre une erreur lors de l'execution de ce petit 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
#!/bin/bash
 
# turn on debug mode
set -x
 
sshd='pidof sshd'
smtpd='pidof snmpd'
 
if [ -z $sshd ]
then
 echo "sshd OK"
else
 echo "sshd down"
fi
 
if [ -z $snmpd ]
then
 echo "snmpd OK"
else
 echo "snmpd down"
fi

c'est juste pour tester si les services sont lancés

comment feriez vous ?

merci d'avance

[becket]

Cela me semble plus court et plus clair :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
service='xterm bash smtpd ssh'
for s in $service
do
  if [  "S$(pidof $s)"  != "S" ]
  then
  echo $s OK
  else
  echo $s ne tourne pas 
  fi
done

[zorooo]

en effet, bien mieux merci

c'est quoi ce "S" et "S$" ?

comment on peut tester dans un if si la valeur d'une variable est null ou si elle a une valeur ?

[becket]

Pourquoi $S et S ? C'est juste une habitude que j'ai

[Invité]

Salut,

Ton erreur dans le script initial réside à ce niveau:

Code :

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

1
2
3
 
sshd='pidof sshd'
smtpd='pidof snmpd'

Tu utilises des single quotes '' au lieu des backquotes `` ou encore $().

Les backquotes ou $() stockent le résultat d'execution d'une commande (pidof <deamon> dans ton cas) tandis que les single quotes stockent la chaine de caractères 'pidof <daemon>' dans ta variable.

[jmelyn]

Bonjour,

Si tu as le petit script /sbin/service dans ta distribution, le test des deux services devient:

Code :

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

1
2
service sshd status
service snmpd status

et j'obtiens chez moi:

Code :

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

1
2
openssh-daemon (pid  1784) is running...
snmpd is stopped

[becket]

Il est plus facile d'utiliser pidof que service à mon avis

[zorooo]

oui je préfère pidof car si le service qui tourne n'as pas de script d'init il passera a trave

[zorooo]

j'ai trouvé plus simple mais y a un petit bug

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
services='httpd xinetd snmpd sshd auditd master exim sendmail master vsftpd'
 
for i in $services
do
if [ -z `pidof $i` ]
then
 echo $i OFF
else
 echo $i ON
fi
done

a cause de :

Code :

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

1
2
3
4
 
++ pidof sshd
+ '[' -z 23950 23948 23411 23409 1540 ']'
./test1.sh: line 10: [: too many arguments

[N_BaH]

Code :

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

if pidof $i >/dev/null; then echo "OK"; else echo "KO"; fi

...
?
les sorties de pidof sont redirigées vers /dev/null, ne reste que le code de retour de la commande...
s'il vaut 0 la condition est remplie
s'il vaut 1 la condition échoue

[jmelyn]

Vu le retour d'erreur, je pencherais plutôt pour un mauvais découpage de la ligne $service. Pourrais-tu ajouter la ligne IFS=" " avant la boucle for? Ce qui force l' "Input Field Separator" à "espace", c'est-à-dire le caractère qui sépare chaque pid dans la variable $services.

[N_BaH]

Bonjour jmelyn,

pas nécessairement...

vois :

Code :

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

1
2
3
4
5
ps -ef | grep sshd
root      2390     1  0 Dec20 ?        00:00:00 /usr/sbin/sshd
root     31451  2390  0 14:16 ?        00:00:00 sshd: nbah [priv]
nbah     31453 31451  0 14:16 ?        00:00:00 sshd: nbah@pts/0 
nbah     31484 31454  0 14:16 pts/0    00:00:00 grep --color=always sshd

...
?

Quel paquet installer pour disposer de ce script service sur CentOS ?

[zorooo]

Quel paquet installer pour disposer de ce script service sur CentOS ?

initscripts-8.45.19.1.EL-1

[zorooo]

avec : IFS=" "
ça ne change rien toujours la meme erreur

je vais utiliser la forme :

Code :

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

if pidof $i >/dev/null; then echo "OK"; else echo "KO"; fi

je ne savais pas que même après un >/dev/null on avait encore le retour de $?

[becket]

Que de complication ....

Code :

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

1
2
 
if [ -z  "`pidof $i`" ]

et le tour est joué

[zorooo]

merci a tous

j'ai 4 solutions pour faire la même chose ça devrait aller