From 6b52336516d8850f8b23ae0a627e6766cc281556 Mon Sep 17 00:00:00 2001 From: Michal Berger Date: Tue, 15 Dec 2020 14:09:02 +0100 Subject: [PATCH] test/vhost: Fix traddr lookup get_traddr() was looking for INI configuration, however, gen_nvme.sh now supports only json flavor. Use jq in such a case to get proper traddr. Change-Id: Ifb5e5e8b3cf3c7b21f67f6129173dcd09cecc6ec Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/5578 Tested-by: SPDK CI Jenkins Reviewed-by: Karol Latecki Reviewed-by: Jim Harris Reviewed-by: Shuhei Matsumoto Community-CI: Mellanox Build Bot --- test/vhost/hotplug/common.sh | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/test/vhost/hotplug/common.sh b/test/vhost/hotplug/common.sh index b7b05ee74..f89b601b4 100644 --- a/test/vhost/hotplug/common.sh +++ b/test/vhost/hotplug/common.sh @@ -208,17 +208,10 @@ function check_disks() { function get_traddr() { local nvme_name=$1 local nvme + nvme="$($rootdir/scripts/gen_nvme.sh)" - while read -r line; do - if [[ $line == *"TransportID"* ]] && [[ $line == *$nvme_name* ]]; then - local word_array=($line) - for word in "${word_array[@]}"; do - if [[ $word == *"traddr"* ]]; then - traddr=$(echo $word | sed 's/traddr://' | sed 's/"//') - fi - done - fi - done <<< "$nvme" + traddr=$(jq -r ".config[] | select(.params.name == \"$nvme_name\") | .params.trtype" <<< "$nvme") + [[ -n $traddr ]] || return 1 } function delete_nvme() {