Commit c2b5b28c authored Oct 05, 2023 by Michal Berger Committed by Tomasz Zawadzki Oct 06, 2023

test/nvme/cuse: Return proper nvme-cli's exit status



We don't use pipefail in our autotest infra so the piped sed here
masks potential failures coming from the nvme-cli. To avoid that,
check nvme-cli's exit status explicitly.

This is cosmetic, but it allows for the tests to fail early on
rather than at the end when the final checks are being made.

Change-Id: I94e975b112da1b216666621b035681ae05374aaf
Signed-off-by: Michal Berger <michal.berger@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20254


Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Karol Latecki <karol.latecki@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

parent 1b8344d6

test/nvme/cuse/spdk_nvme_cli_plugin.sh

+1 −0

Original line number	Diff line number	Diff line
		@@ -21,6 +21,7 @@ nvme() {
		-e 's#-subsys##g' \
		-e 's#PCIE#pcie#g' \
		-e 's#(null)#live#g'
		((PIPESTATUS[0] == 0))
		}

		kernel_out=()