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

test/blobfs: Call process_core() from rocksdb.sh test 



rocksdb.sh is run independently from the autotest.sh as part of the
BlobFS-nightly-autotest job. In case a rogue file lands in the dir
where the results are stored, i.e., a core dump, CI may fail to fetch
the artifacts due to not sufficient permissions. To avoid that, make
sure cores are always processed as part of the test.

Also, adjust permissions across the entire $RESULTS_DIR, just for the
good measure.

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


Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent ea2000cf

test/blobfs/rocksdb/rocksdb.sh

+7 −1
Original line number Diff line number Diff line
@@ -4,6 +4,11 @@ testdir=$(readlink -f $(dirname $0)) 
rootdir=$(readlink -f $testdir/../../..)

source $rootdir/test/common/autotest_common.sh



sanitize_results() {

	process_core

	[[ -d $RESULTS_DIR ]] && chmod 644 "$RESULTS_DIR/"*

}



dump_db_bench_on_err() {

	# Fetch std dump of the last run_step that might have failed

	[[ -e $db_bench ]] || return 0
@@ -74,7 +79,7 @@ $rootdir/scripts/gen_nvme.sh > $ROCKSDB_CONF 
echo "[Global]" >> $ROCKSDB_CONF

echo "TpointGroupMask 0x80" >> $ROCKSDB_CONF



trap 'dump_db_bench_on_err; run_bsdump || :; rm -f $ROCKSDB_CONF; exit 1' SIGINT SIGTERM EXIT

trap 'dump_db_bench_on_err; run_bsdump || :; rm -f $ROCKSDB_CONF; sanitize_results; exit 1' SIGINT SIGTERM EXIT



if [ -z "$SKIP_MKFS" ]; then

	run_test "blobfs_mkfs" $rootdir/test/blobfs/mkfs/mkfs $ROCKSDB_CONF Nvme0n1
@@ -162,3 +167,4 @@ trap - SIGINT SIGTERM EXIT 


run_bsdump

rm -f $ROCKSDB_CONF

sanitize_results