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