From 0b4ccbd68d41c61e300f48d827eeb5d27e0d807f Mon Sep 17 00:00:00 2001 From: Loic Dachary Date: Sat, 18 Oct 2014 17:15:41 -0700 Subject: [PATCH] tests: kill_daemon use $name.pid instead of pidfile So that it can be used instead of stop.sh to stop vstart.sh daemons. The problem with stop.sh is that it kills any daemon, not just a selection. Signed-off-by: Loic Dachary --- src/test/mon/mon-test-helpers.sh | 4 ++-- src/test/osd/osd-test-helpers.sh | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/test/mon/mon-test-helpers.sh b/src/test/mon/mon-test-helpers.sh index deeb9f38f86..9d54bef2246 100644 --- a/src/test/mon/mon-test-helpers.sh +++ b/src/test/mon/mon-test-helpers.sh @@ -60,13 +60,13 @@ function run_mon() { --log-file=$dir/log \ --mon-cluster-log-file=$dir/log \ --run-dir=$dir \ - --pid-file=$dir/pidfile \ + --pid-file=$dir/\$name.pid \ "$@" } function kill_daemons() { local dir=$1 - for pidfile in $(find $dir | grep pidfile) ; do + for pidfile in $(find $dir | grep '\.pid') ; do pid=$(cat $pidfile) for try in 0 1 1 1 2 3 ; do kill -9 $pid 2> /dev/null || break diff --git a/src/test/osd/osd-test-helpers.sh b/src/test/osd/osd-test-helpers.sh index 4b3a8b79541..f754618cc17 100644 --- a/src/test/osd/osd-test-helpers.sh +++ b/src/test/osd/osd-test-helpers.sh @@ -45,7 +45,7 @@ function run_osd() { ceph_args+=" --run-dir=$dir" ceph_args+=" --debug-osd=20" ceph_args+=" --log-file=$dir/osd-\$id.log" - ceph_args+=" --pid-file=$dir/osd-\$id.pidfile" + ceph_args+=" --pid-file=$dir/osd-\$id.pid" ceph_args+=" " ceph_args+="$@" mkdir -p $osd_data