Commit 728af017 authored by Darek Stojaczyk's avatar Darek Stojaczyk Committed by Jim Harris
Browse files

event/nvmf: allow running without [Transport] in the legacy cfg file 



While useful in NVMf target, this is really annoying
in the consolidated spdk target, where NVMe-oF doesn't
even have to be used. OCF tests currently use iscsi_tgt
just because spdk_tgt requires the additional [Transport]
section in the cfg file. Let's remove that requirement.

Change-Id: I418b47d62dcc06b9513f9f0496dc1e39b9d5a554
Signed-off-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/450056


Reviewed-by: default avatarVitaliy Mysak <vitaliy.mysak@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 1468f8dd

lib/event/subsystems/nvmf/conf.c

+1 −5
Original line number Diff line number Diff line
@@ -581,11 +581,7 @@ spdk_nvmf_parse_transports(spdk_nvmf_parse_conf_done_fn cb_fn) 


	/* if we get here, there are no transports defined in conf file */

	free(ctx);

	SPDK_ERRLOG("\nNo valid transport is defined yet.\n"

		    "When using configuration file, at least one valid transport must be defined.\n"

		    "You can refer the [Transport] section in spdk/etc/spdk/nvmf.conf.in as an example.\n");

	cb_fn(-1);



	cb_fn(0);

	return 0;

}