Commit 26f88ce7 authored by Michal Berger's avatar Michal Berger Committed by Konrad Sztyber
Browse files

test: Fail hard when flamegraph.pl is not found 



flamegraph is crucial to represent timing in a more human-readable
fashion so make sure timing_finish() fails hard when it's not found.

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


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
parent ab79d382

test/common/autotest_common.sh

+8 −8
Original line number Diff line number Diff line
@@ -741,14 +741,14 @@ function timing_exit() { 


function timing_finish() {

	flamegraph='/usr/local/FlameGraph/flamegraph.pl'

	if [ -x "$flamegraph" ]; then

	[[ -x "$flamegraph" ]] || return 1



	"$flamegraph" \

		--title 'Build Timing' \

		--nametype 'Step:' \

		--countname seconds \

			$output_dir/timing.txt \

			> $output_dir/timing.svg

	fi

		"$output_dir/timing.txt" \

		> "$output_dir/timing.svg"

}



function create_test_list() {