RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-02-21 01:47:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

qa/standalone/ceph-helpers.sh: Defining custom timeout for wait_for_clean()

Browse Source 
The wait_for_clean() is using the default timeout aka 300sec = 5mn.

wait_for_clean() is trying to find a clean status within that timeout
_or_ reset its counter if any progress got made in between loops.

In a case where the cluster is sane, the recovery should be made in
shorter than 5mn but it the cluster died, waiting for 5mn for nothing is
unefficient.

This patch is about defining a custom timeout for a wait_for_clean() not
to wait much more that 1m30 (90sec). If no progress is made in that
period, there is very few chance this will read the a valid state
anyhow.

Signed-off-by: Erwan Velu <erwan@redhat.com>
This commit is contained in:
Erwan Velu 2018-06-13 14:25:04 +02:00
parent be406bf515
commit 62d2646c30
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
qa/standalone/ceph-helpers.sh
View File

@ -18,6 +18,7 @@
# GNU Library Public License for more details.
#
TIMEOUT=300
WAIT_FOR_CLEAN_TIMEOUT=90
PG_NUM=4
CEPH_BUILD_VIRTUALENV=${TMPDIR:-/tmp}
@ -1387,7 +1388,7 @@ function test_get_timeout_delays() {
##
# Wait until the cluster becomes clean or if it does not make progress
# for $TIMEOUT seconds.
# for $WAIT_FOR_CLEAN_TIMEOUT seconds.
# Progress is measured either via the **get_is_making_recovery_progress**
# predicate or if the number of clean PGs changes (as returned by get_num_active_clean)
#
@ -1396,7 +1397,7 @@ function test_get_timeout_delays() {
function wait_for_clean() {
local num_active_clean=-1
local cur_active_clean
local -a delays=($(get_timeout_delays $TIMEOUT .1))
local -a delays=($(get_timeout_delays $WAIT_FOR_CLEAN_TIMEOUT .1))
local -i loop=0
flush_pg_stats || return 1
@ -1432,7 +1433,7 @@ function test_wait_for_clean() {
run_mon $dir a --osd_pool_default_size=1 || return 1
run_mgr $dir x || return 1
create_rbd_pool || return 1
! TIMEOUT=1 wait_for_clean || return 1
! WAIT_FOR_CLEAN_TIMEOUT=1 wait_for_clean || return 1
run_osd $dir 0 || return 1
wait_for_clean || return 1
teardown $dir || return 1