Double boucle while

**kevin066** · 13/10/2020, 14h34

Bonjour,

Sur 3 serveurs distincts, je souhaite faire un "ls -al" de 2 fichiers "file1" et "file2" que je mets dans un fichier FILE.txt :

Fichier FILE.txt :

Code :

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

1
2
file1
file2

De la même manière je fabrique un fichier SERVER.txt contenant les 3 serveurs en question et j'imagine une boucle while qui permet de lire ce fichier :

Fichier SERVER.txt :

Code :

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

1
2
3
SERVER1
SERVER2
SERVER3

Code :

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

1
2
3
while read Serv; do
     echo $Serv
done < SERVER.txt

Je me sers donc de cette boucle while pour y insérer une seconde boucle permettant de lire FILE.txt pour chaque ligne de SERVER.txt.

Le but est d'aller faire le "ls -al" pour les 2 fichiers sur les 3 serveurs et d'obtenir au final les 3 fichiers :

FILE_SERVER1.txt :

Code :

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

1
2
-rw-r--r-- 1 kevin kevin    21 Oct 13 12:02  file1
-rw-r--r-- 1 kevin kevin    89 Oct 10 04:36  file2

FILE_SERVER2.txt :

Code :

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

1
2
-rw-r--r-- 1 kevin kevin    37 Oct 13 13:10  file1
-rw-r--r-- 1 kevin kevin    45 Oct 10 09:01  file2

FILE_SERVER3.txt :

Code :

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

1
2
-rw-r--r-- 1 kevin kevin    11 Oct 09 14:09  file1
-rw-r--r-- 1 kevin kevin    51 Oct 07 08:30  file2

Pour cela :

Code :

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

1
2
3
4
5
6
7
while read Serv; do
 
        while read File; do
                ssh $Serv 'ls -al $File'
        done < FILE.txt >> FILE_"$Serv".txt
 
done < SERVER.txt

Sauf que je n'obtiens pas la totalité : uniquement 3 fichiers avec le "ls -al" de file1 mais pas du second file2, comme s'il n'y avait qu'un passage pour la seconde boucle.

Quelque chose m'échappe…!

**balkany** · 13/10/2020, 15h28

ssh "$Serv" "ls -al $File" ?

**kevin066** · 13/10/2020, 15h32

Merci BALKANY pour ta réponse rapide,
C'est juste un exemple (pas terrible j'en conviens) mais le souci c'est que la seconde boucle ne s'exécute qu'une fois.

**balkany** · 13/10/2020, 15h41

Je ne vois pas de raison à cela autre que le fait que FILE.txt n'ait qu'une seule ligne, mais en tout cas tu devrais mettre des guillemets doubles comme j'ai fait ci-dessus, car là, ta variable $File n'est pas prise en compte (à moins qu'elle ait une valeur sur la machine distante).

**kevin066** · 13/10/2020, 15h50

Merci.
Le fichier FILE.txt a bien 2 lignes, mais la boucle ne fait qu'un passage pour je ne sais quelle raison.
Il y a donc un souci dans ces 2 boucles imbriquées, la seconde n'est parcourue qu'une seule fois ! Il doit y avoir une subtilité...

**disedorgue** · 13/10/2020, 15h57

Si tu as une liste de 50 fichiers, tu fais 50 connexions ssh sur le même serveur, tu ne trouve pas ça un peu bourrin ?

D'ailleurs, ton souci est à chercher de ce coté là...

**kevin066** · 13/10/2020, 16h03

Merci.

Il faudrait donc pour chaque serveur faire une seul connexion et passer les deux "ls -al" ?

Pour un serveur, les résultats des 2 "ls -al" doivent être l'un à la suite de l'autre comme dans mon message initial.

Donc pas de seconde boucle while ? Mais comment faire ?

Ceci dit je me m'explique pas le fait que dans la seconde boucle il n'y a qu'un passage ?

**Flodelarab** · 13/10/2020, 16h07

Bonjour

On n'utilise pas "ls" dans un script. "ls", c'est pour les humains. Les alternatives sont "find" et "stat".
Tes "while" imbriqués ne fonctionnent pas car ils se disputent l'entrée standard.
Une solution envisageable est l'utilisation de tableau. Mais, attention, ne jamais oublier les double quotes "". Si tes données ont des espaces, c'est la cata, sans double quotes.
Une boucle for sur les tableaux n'aura plus de problème de flux.

Code bash :

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
 
# On prévient que tab est un tableau
declare -a tab
#On remplit le tableau
while IFS='' read ligne
do 
    tab+=("$ligne")
done < file.txt
 
# Cette boucle est un exemple, probablement équivalent à "cat".
for i in "${tab[@]}"
do 
    echo "$i"
done
 
# On vide le tableau
unset tab

Cet exemple ne fait pas d'imbrications. Mais tu sauras faire.

Code :

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

stat -c "%s %n" *

Code :

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

man stat

**disedorgue** · 13/10/2020, 16h12

A la commande ls, tu peux lui passer une liste de fichier et pas uniquement un seul fichier à la fois.

Après, si tu ne veux pas qu'il trie et qu'il donne dans l'ordre de ta liste, essaye avec l'option -f en premier, donc ls -fal "fichier1" "fichier2" "fichier3".

Et suit le conseil de Flodelarab

**kevin066** · 13/10/2020, 16h16

Merci Flodelarab, c'est du high level je vais essayer déjà de comprendre avant de tenter l'imbrication, pas simple.
Cependant, je dois me servir des 2 fichiers FILE.txt et SERVER.txt car il sont destinés à être modifiés sans toucher le code.

Merci Disedorgue. Mais pour passer un nombre indéfini de commande que une seule ligne, à part la boucle.

N'y a t-il pas un moyen de corriger ou modifier mon code initial pour que la seconde boucle soit lue entièrement et non une fois?

**Flodelarab** · 13/10/2020, 20h38

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
while IFS='' read -r ligne
do 
    tabfile+=("$ligne")
done < file.txt
 
while IFS='' read -r ligne
do 
    tabserver+=("$ligne")
done < server.txt
 
for s in "${tabserver[@]}"
do 
    for f in "${tabfile[@]}"
    do 
        echo "Commande sur le serveur $s avec le fichier $f"
    done
done

**disedorgue** · 14/10/2020, 00h51

Pour ne faire qu'une seule connexion par serveur, si je devais reprendre ton script initial, cela ferais un truc qui ressemblerait à ceci (pas testé) :

Code :

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

1
2
3
while read Serv; do
        ssh $Serv 'bash -s' < <(while read File; do echo ls -al \"$File\"; done < File.txt) >FILE_"$Serv".txt 
done < SERVER.txt

ATTENTION, ici le echo n'est pas facultatif.

**kevin066** · 14/10/2020, 13h55

Merci beaucoup pour vos réponses.

Cependant, je m'en suis sorti avec une double boucle for toute simple…

**kevin066** · 14/10/2020, 14h54

Envoyé par disedorgue

Pour ne faire qu'une seule connexion par serveur, si je devais reprendre ton script initial, cela ferais un truc qui ressemblerait à ceci (pas testé) :

Code :

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

1
2
3
while read Serv; do
        ssh $Serv 'bash -s' < <(while read File; do echo ls -al \"$File\"; done < File.txt) >FILE_"$Serv".txt 
done < SERVER.txt

ATTENTION, ici le echo n'est pas facultatif.

Bonjour,

Super, j'ai testé le "ls -al" et ca fonctionne.

Par contre, à la place du "ls -al" on pourrait imaginer un "find" … et là il veut pas :

Code :

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

do echo /bin/find \"$File\" -type f -printf '%s %p\n'

/bin/find: paths must precede expression: %pn

SOLUTION : mettre la commande entre guillemets

Code :

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

do echo "/bin/find \"$File\" -type f -printf '%s %p\n'"

Comme je disais dans mon message précédent, avant de tester cette solution je m'en étais sorti avec une double boucle for et ça fonctionne même si c'est pas prévu pour faire ce genre de chose :

Code :

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

1
2
3
4
5
for File in $(cat FILE.txt); do
    for Serv in $(cat SERVER.txt); do
       ssh $Serv 'ls -al "$File"' > FILE_"$Serv".txt
    done
done

**disedorgue** · 14/10/2020, 15h27

si, ça fonctionne si tu fais ce que dit le message d'erreur ...

Je suppose que tu te connectes sur des AIX ou Solaris ?

De manière générale, find prend un chemin depuis lequel il doit chercher le pattern qui peut être un nom de fichier:

Code :

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