From 14671b8f4e9e0c1c9b38ff3d42fc0e89e802144e Mon Sep 17 00:00:00 2001 From: Michal Berger Date: Thu, 19 Jan 2023 12:50:56 +0100 Subject: [PATCH] test/fuzz: Don't gather coverage under short-fuzz job That was the main intention behind the following patch: https://review.spdk.io/gerrit/c/spdk/spdk/+/15356 however, the introduced logic was faulty. Fix it. Signed-off-by: Michal Berger Change-Id: I84ec7ba79d477756c9064e592e93140e68038bc3 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16348 Tested-by: SPDK CI Jenkins Reviewed-by: Tomasz Zawadzki Reviewed-by: Karol Latecki Reviewed-by: Konrad Sztyber --- test/fuzz/llvm.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/fuzz/llvm.sh b/test/fuzz/llvm.sh index a0070c96f..2c3410d88 100755 --- a/test/fuzz/llvm.sh +++ b/test/fuzz/llvm.sh @@ -55,7 +55,7 @@ function lcov_stop() { # Collect coverage data when run fuzzers for longer period of time # this allow to check coverage progression between runs, and grow of corpus files -if [[ $SPDK_TEST_FUZZER -eq 1 || $SPDK_TEST_FUZZER_SHORT -eq 0 ]]; then +if [[ $SPDK_TEST_FUZZER_SHORT -eq 0 ]]; then lcov_start fi @@ -66,7 +66,7 @@ for fuzzer in "${fuzzers[@]}"; do esac done -if [[ $SPDK_TEST_FUZZER -eq 1 || $SPDK_TEST_FUZZER_SHORT -eq 0 ]]; then +if [[ $SPDK_TEST_FUZZER_SHORT -eq 0 ]]; then lcov_stop genhtml $llvm_out/cov_total.info --output-directory $llvm_out fi