diff options
author | Mohit Agrawal <moagrawal@redhat.com> | 2020-03-12 21:12:13 +0530 |
---|---|---|
committer | MOHIT AGRAWAL <moagrawa@redhat.com> | 2020-03-20 04:08:42 +0000 |
commit | fb20713b380e1df8d7f9e9df96563be2f9144fd6 (patch) | |
tree | cd61da00b382c7b805e74d5e9370e11df2718f49 /run-tests.sh | |
parent | 5410cc35ea09b1932c8eae4933fbf84f462e61ea (diff) |
Posix: Use simple approach to close fd
Problem: posix_release(dir) functions add the fd's into a ctx->janitor_fds
and janitor thread closes the fd's.In brick_mux environment it is
difficult to handle race condition in janitor threads because brick
spawns a single janitor thread for all bricks.
Solution: Use synctask to execute posix_release(dir) functions instead of
using background a thread to close fds.
Credits: Pranith Karampuri <pkarampu@redhat.com>
Change-Id: Iffb031f0695a7da83d5a2f6bac8863dad225317e
Fixes: bz#1811631
Signed-off-by: Mohit Agrawal <moagrawal@redhat.com>
Diffstat (limited to 'run-tests.sh')
-rwxr-xr-x | run-tests.sh | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/run-tests.sh b/run-tests.sh index d3ff78d6915..2768adb5afd 100755 --- a/run-tests.sh +++ b/run-tests.sh @@ -357,8 +357,7 @@ function run_tests() selected_tests=$((selected_tests+1)) echo echo $section_separator$section_separator - if [[ $(get_test_status $t) == "BAD_TEST" ]] || \ - [[ $(get_test_status $t) == "BRICK_MUX_BAD_TEST" ]] && \ + if [[ $(get_test_status $t) =~ "BAD_TEST" ]] && \ [[ $skip_bad_tests == "yes" ]] then skipped_bad_tests=$((skipped_bad_tests+1)) |