Commit c7385758 authored by Michal Berger's avatar Michal Berger Committed by Jim Harris
Browse files

autotest: Move nvmf's fuzzing outside the main suite 



This test is static in nature and not dependent on actual nvmf setup
that comes from the CI - everything is driven via a static json config
that comes with the test itself.

Instead, keep fuzzing directly in autotest.sh, under a top dedicated
test flag SPDK_TEST_FUZZER - this should allow users to run only
fuzzing part without touching the majority of the nvmf suite.

Change-Id: I9c33c46e05f7c4b977b041673fe200d9d2989265
Signed-off-by: default avatarMichal Berger <michal.berger@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/14287


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarPawel Piatek <pawelx.piatek@intel.com>
parent 044bb175

autotest.sh

+6 −0
Original line number Diff line number Diff line
@@ -342,6 +342,12 @@ if [ $SPDK_RUN_FUNCTIONAL_TEST -eq 1 ]; then 
	if [[ $SPDK_TEST_SMA -eq 1 ]]; then

		run_test "sma" ./test/sma/sma.sh

	fi



	if [[ $SPDK_TEST_FUZZER -eq 1 ]]; then

		# TODO: Consider re-using test/fuzz directory for all fuzzing tests with proper

		# structure, e.g.: test/fuzz/nvmf/*, test/fuzz/vfio/*, etc.

		run_test "nvmf_llvm_fuzz" test/nvmf/target/llvm_nvme_fuzz.sh

	fi

fi



timing_enter cleanup

test/nvmf/nvmf.sh

+0 −4
Original line number Diff line number Diff line
@@ -13,10 +13,6 @@ trap "exit 1" SIGINT SIGTERM EXIT 


TEST_ARGS=("$@")



if [[ $SPDK_TEST_FUZZER -eq 1 ]]; then

	run_test "nvmf_llvm_fuzz" test/nvmf/target/llvm_nvme_fuzz.sh "${TEST_ARGS[@]}"

fi



if [[ $SPDK_TEST_URING -eq 0 ]]; then

	run_test "nvmf_example" test/nvmf/target/nvmf_example.sh "${TEST_ARGS[@]}"

	run_test "nvmf_filesystem" test/nvmf/target/filesystem.sh "${TEST_ARGS[@]}"