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

scripts/rxe_cfg_small: Print rxe-net devs only if list is non-empty



Otherwise it may cause false-positives, depending on how the output
is gathered, e.g. this:

  $ mapfile -t rxe_net < <(rxe_cfg_small.sh rxe-net)

will create rxe_net[@] with a single, empty item, fooling user that
there are some devices present even though they are not.

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


Reviewed-by: default avatarKarol Latecki <karol.latecki@intel.com>
Reviewed-by: default avatarJim Harris <jim.harris@gmail.com>
Reviewed-by: default avatarKamil Godzwon <kamilx.godzwon@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 7662387c
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -313,7 +313,7 @@ case "${1:-status}" in
		status | grep -E "${match:-.}"
		;;
	rxe-net)
		printf '%s\n' "${rxe_to_net[@]}"
		((${#rxe_to_net[@]} > 0)) && printf '%s\n' "${rxe_to_net[@]}"
		;;
	*)
		printf 'Invalid argument (%s)\n' "$1"