From 9c2b9f445f827121207cba6b2128a988833835c5 Mon Sep 17 00:00:00 2001
From: Michal Berger <michalx.berger@intel.com>
Date: Wed, 20 Oct 2021 12:38:19 +0200
Subject: [PATCH] test/dd: Return from gen_conf() if no configuration was
 provided

Signed-off-by: Michal Berger <michalx.berger@intel.com>
Change-Id: I9bbd8ad2d7e369959c27433905749adab2958d07
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9939
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Karol Latecki <karol.latecki@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
---
 test/dd/common.sh | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/test/dd/common.sh b/test/dd/common.sh
index 1475274ec..5c63c1eba 100644
--- a/test/dd/common.sh
+++ b/test/dd/common.sh
@@ -38,6 +38,10 @@ gen_conf() {
 	# of the values is done here. extra_subsystems[] can store extra
 	# json configuration for different subsystems, other than bdev.
 
+	if (($# == 0)) && [[ -z ${!method_@} ]]; then
+		return 1
+	fi
+
 	methods=("${@:-${!method_@}}")
 	local IFS=","