Commit 03f2331b authored by Artur Paszkiewicz's avatar Artur Paszkiewicz Committed by Konrad Sztyber
Browse files

test/raid: remove some redundant jq calls 



Signed-off-by: default avatarArtur Paszkiewicz <artur.paszkiewicz@intel.com>
Suggested-by: default avatarMichal Berger <michal.berger@intel.com>
Change-Id: I636489a24b1052772fdcb92d7f33ae05bcd840f4
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24576


Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz@tzawadzki.com>
parent 3b3688f8

test/bdev/bdev_raid.sh

+4 −6
Original line number Diff line number Diff line
@@ -180,19 +180,17 @@ function verify_raid_bdev_process() { 
function verify_raid_bdev_properties() {

	local raid_bdev_name=$1

	local raid_bdev_info

	local base_bdev_info

	local base_bdev_names

	local name

	local cmp_raid_bdev cmp_base_bdev



	raid_bdev_info=$($rpc_py bdev_get_bdevs -b $raid_bdev_name | jq '.[]')

	base_bdev_names=$(jq -r '.driver_specific.raid.base_bdevs_list[] | select(.is_configured == true).name' <<< "$raid_bdev_info")

	cmp_raid_bdev=$(jq -r '[.block_size, .md_size, .md_interleave, .dif_type] | join(" ")' <<< "$raid_bdev_info")



	for name in $base_bdev_names; do

		base_bdev_info=$($rpc_py bdev_get_bdevs -b $name | jq '.[]')

		[[ $(jq '.block_size' <<< "$raid_bdev_info") == $(jq '.block_size' <<< "$base_bdev_info") ]]

		[[ $(jq '.md_size' <<< "$raid_bdev_info") == $(jq '.md_size' <<< "$base_bdev_info") ]]

		[[ $(jq '.md_interleave' <<< "$raid_bdev_info") == $(jq '.md_interleave' <<< "$base_bdev_info") ]]

		[[ $(jq '.dif_type' <<< "$raid_bdev_info") == $(jq '.dif_type' <<< "$base_bdev_info") ]]

		cmp_base_bdev=$($rpc_py bdev_get_bdevs -b $name | jq -r '.[] | [.block_size, .md_size, .md_interleave, .dif_type] | join(" ")')

		[[ "$cmp_raid_bdev" == "$cmp_base_bdev" ]]

	done

}