Bash скрипт ждет процессов и получает код возврата

13

Я пытаюсь создать скрипт, который запустит много фоновых команд. Для каждой фоновой команды мне нужно получить код возврата.

Я пробовал следующий скрипт:

 #!/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]}

Моя проблема во время цикла ожидания: если второй pid завершится раньше первого, я не смогу получить код возврата.

Я знаю, что могу запустить wait pid1 pid2, но с помощью этой команды я не могу получить код возврата всех команд.

Есть идеи ?

Хьюго
источник

Ответы:

6

Вы можете сделать это с помощью временного каталога.

# 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"
Крис Даун
источник
9

Проблема больше с вашим

for i in ${#pid[@]}

Который есть for i in 2.

Скорее должно быть:

for i in 1 2

или

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

wait "$pid" будет возвращать код завершения работы с bash(и POSIX оболочки, но не zsh) , даже если работа уже завершается , когда waitбыла начата.

Стефан Шазелас
источник
5

Общая реализация без временных файлов.

#!/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"
Джон Нэлли
источник
Спасибо :-) Это именно то, что я искал!
Qorbani
0

Ответ Стефана хорош, но я бы предпочел

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

который будет перебирать ключи pidмассива, независимо от того, какие записи все еще существуют, так что вы можете адаптировать его, выйти из цикла и перезапустить весь цикл, и он будет просто работать. И вам не нужны последовательные значения iдля начала.

Конечно, если вы имеете дело с тысячами процессов, то, возможно, подход Сепана будет несколько более эффективным, если у вас есть непрозрачный список.

Мартин Кили
источник