Commit 4ceace76 authored by Michal Berger's avatar Michal Berger Committed by Konrad Sztyber
Browse files

test/common: Make sure get_nvmes_with_ns_management() doesn't report failure 



This function is meant to simply list nvme devices which support
ns management, it shouldn't trigger errexit (it's up to the caller to
verify returned list of devices).

Currently, this function may report failure in case last ctrl from
the list doesn't have proper support enabled - this may be problematic
in case there's a mix of different nvmes installed with different
capabilities.

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


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>
parent 73c43eee

test/common/nvme/functions.sh

+2 −0
Original line number Diff line number Diff line
@@ -147,6 +147,8 @@ get_nvmes_with_ns_management() { 
	for ctrl in "${!ctrls[@]}"; do

		get_oacs "$ctrl" nsmgt && echo "$ctrl"

	done



	return 0

}



get_nvme_with_ns_management() {