Commit 4d00fb20 authored by Boris Glimcher's avatar Boris Glimcher Committed by Tomasz Zawadzki
Browse files

python/cli: nvmf_set_config do not send empty list 



if '-p|--passthru-admin-cmds' is missing, code was still
sending empty list to the spdk app, now this is fixed

Change-Id: Iee4cc16ae7f83bbf3bb54fd24d5e90d34f393064
Signed-off-by: default avatarBoris Glimcher <Boris.Glimcher@emc.com>
Reviewed-on: https://review.spdk.io/c/spdk/spdk/+/26953


Reviewed-by: default avatarJacek Kalwas <jacek.kalwas@nutanix.com>
Tested-by: default avatarSPDK Automated Test System <spdkbot@gmail.com>
Reviewed-by: default avatarJim Harris <jim.harris@nvidia.com>
parent 6538fbde

python/spdk/cli/nvmf.py

+3 −1
Original line number Diff line number Diff line
@@ -23,7 +23,9 @@ def add_parser(subparsers): 
    def nvmf_set_config(args):

        all_admin_cmd_passthru = ('identify_ctrlr', 'identify_uuid_list', 'get_log_page', 'get_set_features', 'sanitize',

                                  'security_send_recv', 'fw_update', 'nvme_mi', 'vendor_specific')

        if 'all' in args.passthru_admin_cmds:

        if not args.passthru_admin_cmds:

            admin_cmd_passthru = None

        elif 'all' in args.passthru_admin_cmds:

            admin_cmd_passthru = {cmd: True for cmd in all_admin_cmd_passthru}

        else:

            admin_cmd_passthru = {cmd: True for cmd in args.passthru_admin_cmds}