From 92b432139da587c85648ac96dc31ed8c4b5d7b97 Mon Sep 17 00:00:00 2001 From: Brian Behlendorf Date: Thu, 13 Sep 2018 13:35:09 -0700 Subject: [PATCH] Add removal_resume_export to zts-report.py Add the removal_resume_export test case to the possible failure section of the zts-report.py and reference the Github issue. In the CI environment this test has proven to be unreliable due to the way it detects the removal thread. This is a flaw in the test and not device removal so update the result summary accordingly. Additionally, increase the allowed timeout in an effort to reduce the observed rate of false positves. Reviewed-by: George Melikov Signed-off-by: Brian Behlendorf Closes #7895 Issue #7894 --- tests/test-runner/bin/zts-report.py | 1 + .../tests/functional/removal/removal_resume_export.ksh | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/test-runner/bin/zts-report.py b/tests/test-runner/bin/zts-report.py index 5747f1422a..61cbb54ddc 100755 --- a/tests/test-runner/bin/zts-report.py +++ b/tests/test-runner/bin/zts-report.py @@ -256,6 +256,7 @@ maybe = { 'no_space/enospc_002_pos': ['FAIL', enospc_reason], 'projectquota/setup': ['SKIP', exec_reason], 'redundancy/redundancy_004_neg': ['FAIL', '7290'], + 'removal/removal_resume_export': ['FAIL', '7894'], 'reservation/reservation_008_pos': ['FAIL', '7741'], 'reservation/reservation_018_pos': ['FAIL', '5642'], 'rsend/rsend_019_pos': ['FAIL', '6086'], diff --git a/tests/zfs-tests/tests/functional/removal/removal_resume_export.ksh b/tests/zfs-tests/tests/functional/removal/removal_resume_export.ksh index 5cecfdb5dd..2c69c591af 100755 --- a/tests/zfs-tests/tests/functional/removal/removal_resume_export.ksh +++ b/tests/zfs-tests/tests/functional/removal/removal_resume_export.ksh @@ -53,7 +53,7 @@ function ensure_thread_running # spa_address if is_linux; then typeset TRIES=0 typeset THREAD_PID - while [[ $TRIES -lt 10 ]]; do + while [[ $TRIES -lt 50 ]]; do THREAD_PID=$(pgrep spa_vdev_remove) [[ "$THREAD_PID" ]] && break sleep 0.1