Commit c6db5f41 authored by Michal Berger's avatar Michal Berger Committed by Tomasz Zawadzki
Browse files

autotest: Don't run spdk_dd tests under crypto job 



Currently, crypto job also executes tests from the SPDK_TEST_BLOCKDEV
suite, including spdk_dd tests. The dd tests under a bare-metal system
can take up to 4 minutes alone and that brings the crypto job to the
very edge of timing out.

Since these tests are extensively covered under separate vg jobs we
won't lose any coverage by excluding them from crypto's - the basic
set of SPDK_TEST_BLOCKDEV tests would still remain as part of this
job.

Fixes: #2017

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


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarPawel Piatek <pawelx.piatek@intel.com>
Reviewed-by: default avatarKarol Latecki <karol.latecki@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarKrzysztof Karas <krzysztof.karas@intel.com>
parent a82e8478

autotest.sh

+8 −1
Original line number Diff line number Diff line
@@ -190,9 +190,16 @@ if [ $SPDK_RUN_FUNCTIONAL_TEST -eq 1 ]; then 


	if [[ $(uname -s) == Linux ]]; then

		if [[ $SPDK_TEST_BLOCKDEV -eq 1 || $SPDK_TEST_URING -eq 1 ]]; then

			# The crypto job also includes the SPDK_TEST_BLOCKDEV in its configuration hence the

			# dd tests are executed there as well. However, these tests can take a significant

			# amount of time to complete (up to 4min) on a physical system leading to a potential

			# job timeout. Avoid that by skipping these tests - this should not affect the coverage

			# since dd tests are still run as part of the vg jobs.

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

				run_test "spdk_dd" test/dd/dd.sh

			fi

		fi

	fi



	if [ $SPDK_TEST_JSON -eq 1 ]; then

		run_test "test_converter" test/config_converter/test_converter.sh