web-dev-qa-db-ja.com

Bashスクリプトはプロセスを待機して戻りコードを取得します

多くのバックグラウンドコマンドを開始するスクリプトを作成しようとしています。バックグラウンドコマンドごとに、戻りコードを取得する必要があります。

私は次のスクリプトを試しています:

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

私の問題は待機ループ中にあり、2番目のPIDが最初のPIDの前に終了した場合、戻りコードを取得できません。

Wait pid1 pid2を実行できることはわかっていますが、このコマンドではすべてのコマンドの戻りコードを取得できません。

何か案が ?

13
Hugo

これは、一時ディレクトリを使用して行うことができます。

# 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"
6
Chris Down

問題はあなたにもっとあります

for i in ${#pid[@]}

for i in 2

それはむしろ:

for i in 1 2

または

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

wait "$pid"willbashの開始時にジョブがすでに終了している場合でも、zsh(およびwaitではなくPOSIXシェル)を使用してジョブの終了コードを返します。

9

一時ファイルを使用しない一般的な実装

#!/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"
5
Jon Nalley

ステファンの答えは良いですが、私は好むでしょう

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

これは、どのエントリがまだ存在するかに関係なく、pid配列のキーを反復処理するので、それを適応させ、ループを抜け出し、ループ全体を再起動すれば、うまくいきます。そして、最初にiの連続する値は必要ありません。

もちろん、何千ものプロセスを処理している場合、非スパースリストがある場合、おそらくステパンのアプローチのほうがはるかに効率的です。

0
Martin Kealey