Discussion :

[Bbastou]

Bonjours a vous !,

(je m'excuse si ne publie pas correctement sur le forum c'est ma première fois)

j’aimerai tout bonnement faire un script shell qui m'envoie par mail tout les 24h avec dedans : un état d'un site internet et si le serveur est up ou encore le temps de chargement de la page.

En ce qui concerne la gestion du ping, la création d'un tableau pour stocker les variables et l'envoi d'un mail tout fonctionne correctement dans des script séparée, sauf qu'il faut que je regroupe tout en un.

c'est la que cela coinces par exemple avec le code ci-joint le script n’exécute pas commande (ou variable ) pong, il reste bloquer sur le premier echo et utilise toutes la ressources cpu.

Je suis (malheureusement) débutant en script, je ne vois pas ce qui cloche .

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
 
#!/bin/bash
 
rep=1
let "testHeure=0"
 
 
#fonction qui fait que le script tourne en boucle 
function tourneEnBoucle {
    echo -e "\n"
}
 
 
 
function ping  {
    heure=$(date +%Hh%M)
    echo "Est ce que je rentre dans le ping"
 
    ########LE PING########BASTIEN
 
    pong=$(ping -c 2 -q -w 1 xx.xx.xx.xx|grep rtt | cut -d "/" -f5 | cut -c 1-2);
    #Le ping échoue quand le temps de rep dépasse 1s .
 
    echo -e "$(date): ttr $pong ms\n"; 
    #Affiche l'heure et la date en plus de la valeur du temps de ping 
 
 
    # Defini si la variable pong est nul ou non, si non lance le chargement de la page :
    if [ -n "$pong" ] #si le ping est ok, l'on passe au chargement du site web.
    then 
        tempsLoadingSite=$(curl -m 2 -s -o /dev/null -w '%{time_total}\n' https://www.xxx.fr);
 
        # le curl charge le site en mode silencieux -m 2 timeout 2s , le -o l'enregistre dans la "poubelle" puis -w (write out)qui va nous afficher 
        #la varaible time_total en secondes
 
 
        echo "le site est good, $tempsLoadingSite s de chargement  "
    else
        echo "Le ping ne fonctionne pas "
    fi
 
    ########LE PING########
 
 
    if [ $heure = "16h56" ] && [ $testHeure -eq 1 ]
    then
        let "testHeure=0"
    fi
 
    sleep 5
}
 
 
 
while [ "$rep" -eq 1 ]
do
    ping
    tourneEnBoucle
done

Un grand merci pour votre aide ou pour votre explication !

[ctac_]

Salut,
J'ai pas trop regardé et je peux me tromper mais ...
ping {
pong= $(ping ...)
}
on relance ping ...
pong=$(ping ...)
...
Je pense que les noms des fonctions sont bien choisies.
Cordialement.

[Flodelarab]

Bonjour

#fonction qui fait que le script tourne en boucle
function tourneEnBoucle {
echo -e "\n"
}

Non. Ce qui fait que ça tourne en boucle, ce n'est pas cette fonction anodine, mais la boucle infinie immonde qui est à la fin du script. Aucun programme informatique au monde n'a jamais tourné à l'infini. Alors mets une condition d'arrêt crédible. Ou mieux, enlève cette boucle inutile.

function ping {

Pas très malin d'appeler une fonction comme une commande. Regarde ma console :

Code :

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

1
2
3
4
5
6
$ function ping { echo ok;}
$ ping
ok
$ ping -c 2 -q -w 1 google.com
ok
$

Tu vois le problème ? Quand j'ai voulu pinguer Google, il a cru que j'appelais la fonction et non la commande et rien n'a marché.


Enfin, voici un site pour vérifier la syntaxe de ton code :
https://www.shellcheck.net/

[Bbastou]

Re-Bonsoir,

Donc j'ai bien vérifié le code avec le shellCheck d’après le site c'est ok pour l'instant, j'ai aussi modifié le nom de la fonction ping par faitping et cela fonctionne top merci beaucoup ! ,

Par contre je veux bien que tu m'expliques (ou un lien web) pour dégager la boucle infinie ^^ car je voudrais juste que le script fasse la boucle faitping toutes les 15 min en gros, et que après la vérification de l’heure il envoie un mail.

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
#!/bin/bash
 
rep=1
((testHeure=0))
 
 
#fonction qui fait que le scrit tourne en boucle function tourneEnBoucle
 
 
 
function faitping  {
	heure=$(date +%Hh%M)
	echo "Est ce que je rentre dans le ping"
 
	########LE PING########BASTIEN ET KAELIG
 
	pong=$(ping -c 2 -q -w 1 195.83.167.102 |grep rtt | cut -d "/" -f5 | cut -c 1-2);
	#Le ping échoue quand le temps de rep dépasse 1s .
 
	echo -e "$(date): ttr $pong ms\n"; 
	#Affiche l'heure et la date en plus de la valeur du temps de ping 
 
 
	# Defini si la variable pong est nul ou non, si non lance le chargement de la page :
	if [ -n "$pong" ] #si le ping est ok, l'on passe au chargement du site web.
	then 
		tempsLoadingSite=$(curl -m 2 -s -o /dev/null -w '%{time_total}\n' https://www.ac-nantes.fr);
 
		# le curl charge le site en mode silencieux -m 2 timeout 2s , le -o l'enregistre dans la "poubelle" puis -w (write out)qui va nous afficher 
		#la varaible time_total en secondes
		#tsite2=$(time wget -E -H -k -K -p -nv -q https://www.ac-nantes.fr | awk '{print $3,$4}'); Non fonctionelle pour nous  
 
		echo "le site est good, $tempsLoadingSite s de chargement ヾ(≧∇≦)ゞ "
	else
		echo "Le ping ne fonctionne pas ╥﹏╥ "
	fi
 
	########LE PING########
 
 
	if [ "$heure" = "21h51" ] && [ "$testHeure" -eq 1 ]
  echo "$testHeure "
    then
		((testHeure=0))
 
	fi
 
	sleep 5
}
 
 
 
while [ "$rep" -eq 1 ]
do
	faitping
 
done

[Ti-Slackeux]

Bonsoir,

Juste comme ça, mais un script qui fait le boulot une fois et s'arrete c'est moins de soucis non ?
après un cron pour le faire tourner toutes les 24 heures ça devrait faire l'affaire ?

hth,

[Bbastou]

Bonsoir,

Oui j'y avais pensé mais pas possible d'utiliser cron tab ici , sinon oui cela aurait été beaucoup plus simple effectivement!

[Flodelarab]

Ti-Slackeux a raison. C'est le boulot de cron.

Sinon :

Code :

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

man sleep

Quant à la boucle infinie, je vois bien que tu initialises $rep, que tu testes $rep, mais je ne vois pas où tu modifies rep afin d'arrêter la boucle. Ai-je la berlue ?

[Invité]

pas possible d'utiliser crontab ici

et pourquoi cela ?

[disedorgue]

Bon, déjà, pourquoi un ping puis un curl ensuite puisque de toute façon, ce que tu récupères en cas de succès c'est la durée de chargement de la page ?
A la rigueur, un curl puis un ping si la durée du curl ne convient pas pour voir si c'est réseau ou appli serait plus logique: perso, je ne ferais que le curl, car dans tous les cas, il doit y avoir intervention si problème...

Et mon coup de gueule:
Si pas droit au cron, tu précises à ta direction que le script est fait, il n'y a plus qu'à le lancer aux heures que vous désirez et tu retires ta boucle infinie ou tout subterfuge qui pourrait remplacer un outil tel que cron.

Des fois, faut arrêter d'écouter des abrutis qui décident entre autre que cron c'est pas bien mais ne proposent rien en retour.

Et d'ailleurs, tu auras l'air bien malin, si pour une raison ou une autre, ton script s'arrête... comment tu le détectes ?

Et quand tu auras une centaine de scripts de la sorte, imagine le bordel...

cron n'est pas parfait, mais faute de mieux, en général, il fait le job.

[Invité]

un admin peut interdire cron, mais peut-il interdire l'instance utilisateur de systemd, ou simplement n'est-ce pas configuré par défaut ?
à condition bien sûr que systemd soit installé !

[gangsoleil]

Bonjour,

Je suis d'accord avec ce qui a été dit précédemment, que je récapitule donc :
- c'est le boulot de cron, systemd ou un autre outil du genre (ordonnanceur ?)de lancer un script régulièrement, donc tu as intérêt à te baser là dessus. Comme évoqué, comment vas-tu faire pour savoir si ton programme tourne toujours ? Tu fais un autre script, qui boucle aussi, pour faire un ps toutes les N secondes ? Et ce même script, qui le surveille ?
- Si vraiment tu ne peux pas utiliser un de ces outils, mets un "sleep N" dans ton code. L'attente active bouffe le CPU, donc c'est à proscrire. Clairement, ce n'est pas la bonne solution, mais c'est déjà mieux
- Pourquoi ping puis curl ? Un appel à curl tout seul ne te suffit pas ? Et pourtant, si le site distant tombe entre le ping et le curl, c'est bien le même cas non ?
- La variable rep ne sert à rien dans ton script. Ce que tu as écrit revient à "while true", car rep n'est jamais modifiée. À toi de trouver des cas de sortie.