Skrypt Bash czeka na procesy i otrzymuje kod powrotu

13

Próbuję utworzyć skrypt, który uruchomi wiele poleceń w tle. Do każdego polecenia w tle muszę uzyskać kod powrotu.

Próbowałem następującego skryptu:

 #!/bin/bash
set -x
pid=()
return=()


for i in 1 2
do
 echo start $i
 ssh mysql "/root/test$i.sh" &
 pid[$i]=$!
done

for i in ${#pid[@]}
do
echo ${pid[$i]}
wait ${pid[$i]}
return[$i]=$?

if [ ${return[$i]} -ne 0 ]
then
  echo mail error
fi

done

echo ${return[1]}
echo ${return[2]}

Mój problem dotyczy pętli oczekiwania, jeśli drugi pid zakończy się przed pierwszym, nie będę w stanie uzyskać kodu powrotu.

Wiem, że mogę uruchomić czekaj pid1 pid2, ale dzięki temu poleceniu nie mogę uzyskać kodu powrotu wszystkich poleceń.

Dowolny pomysł ?

Hugo
źródło

Odpowiedzi:

6

Możesz to zrobić za pomocą katalogu tymczasowego.

# Create a temporary directory to store the statuses
dir=$(mktemp -d)

# Execute the backgrouded code. Create a file that contains the exit status.
# The filename is the PID of this group's subshell.
for i in 1 2; do
    { ssh mysql "/root/test$i.sh" ; echo "$?" > "$dir/$BASHPID" ; } &
done

# Wait for all jobs to complete
wait

# Get return information for each pid
for file in "$dir"/*; do
    printf 'PID %d returned %d\n' "${file##*/}" "$(<"$file")"
done

# Remove the temporary directory
rm -r "$dir"
Chris Down
źródło
9

Problem dotyczy bardziej twojego

for i in ${#pid[@]}

Co jest for i in 2.

Powinno to raczej być:

for i in 1 2

lub

for ((i = 1; i <= ${#pid[@]}; i++))

wait "$pid" będzie zwróci kod zakończenia pracy z bash(i muszle POSIX, ale nie zsh), nawet jeśli zadanie już zakończone, gdy waitzostała uruchomiona.

Stéphane Chazelas
źródło
5

Ogólna implementacja bez plików tymczasowych.

#!/usr/bin/env bash

## associative array for job status
declare -A JOBS

## run command in the background
background() {
  eval $1 & JOBS[$!]="$1"
}

## check exit status of each job
## preserve exit status in ${JOBS}
## returns 1 if any job failed
reap() {
  local cmd
  local status=0
  for pid in ${!JOBS[@]}; do
    cmd=${JOBS[${pid}]}
    wait ${pid} ; JOBS[${pid}]=$?
    if [[ ${JOBS[${pid}]} -ne 0 ]]; then
      status=${JOBS[${pid}]}
      echo -e "[${pid}] Exited with status: ${status}\n${cmd}"
    fi
  done
  return ${status}
}

background 'sleep 1 ; false'
background 'sleep 3 ; true'
background 'sleep 2 ; exit 5'
background 'sleep 5 ; true'

reap || echo "Ooops! Some jobs failed"
Jon Nalley
źródło
Dziękuję :-) Właśnie tego szukałem!
Qorbani,
0

Odpowiedź Stéphane'a jest dobra, ale wolałbym

for i in ${!pid[@]}
do
    wait ${pid[i]}
    return[i]=$?
    unset "pid[$i]"
done

który będzie iterował po kluczach pidtablicy, bez względu na to, które wpisy nadal istnieją, więc możesz je dostosować, zerwać z pętli i ponownie uruchomić całą pętlę, a ona po prostu zadziała. Na ipoczątek nie potrzebujesz kolejnych wartości .

Oczywiście, jeśli masz do czynienia z tysiącami procesów, być może podejście Stépane'a byłoby frakcjonalnie bardziej wydajne, jeśli masz nierzadką listę.

Martin Kealey
źródło