]> granicus.if.org Git - zfs/commitdiff
Add removal_resume_export to zts-report.py
authorBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 13 Sep 2018 20:35:09 +0000 (13:35 -0700)
committerGitHub <noreply@github.com>
Thu, 13 Sep 2018 20:35:09 +0000 (13:35 -0700)
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 <mail@gmelikov.ru>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #7895
Issue #7894

tests/test-runner/bin/zts-report.py
tests/zfs-tests/tests/functional/removal/removal_resume_export.ksh

index 5747f1422a30c841dec840182f594c1f512bc026..61cbb54ddccba3725de61eeb75f6da63e4ef02b1 100755 (executable)
@@ -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'],
index 5cecfdb5dd57528ef0e0f77cc03ed33f5c6677bf..2c69c591af84846b15d311b09a1c388cac95899b 100755 (executable)
@@ -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