Commit 6a35d0fd authored by Pawel Kaminski's avatar Pawel Kaminski Committed by Jim Harris
Browse files

spdkcli: Skip refreshing node if spdkcli is run noninteractive 



Change-Id: I38662ce05acbf02092b1f02c72800aaf8f448136
Signed-off-by: default avatarPawel Kaminski <pawelx.kaminski@intel.com>
Reviewed-on: https://review.gerrithub.io/c/445012


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarPawel Wodkowski <pawelx.wodkowski@intel.com>
Reviewed-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent e028ae45

scripts/spdkcli.py

+2 −0
Original line number Diff line number Diff line
@@ -32,6 +32,7 @@ def main(): 
    :return:

    """

    spdk_shell = ConfigShell("~/.scripts")

    spdk_shell.interactive = True

    add_quotes_to_shell(spdk_shell)



    parser = argparse.ArgumentParser(description="SPDK command line interface")
@@ -51,6 +52,7 @@ def main(): 


    if len(args.commands) > 0:

        try:

            spdk_shell.interactive = False

            spdk_shell.run_cmdline(" ".join(args.commands))

        except Exception as e:

            sys.stderr.write("%s\n" % e)

scripts/spdkcli/ui_node.py

+8 −7
Original line number Diff line number Diff line
@@ -43,7 +43,8 @@ class UINode(ConfigNode): 
            self.shell.log.debug("Command %s succeeded." % command)

            return result

        finally:

            if command in ["create", "delete", "delete_all", "add_initiator",

            if self.shell.interactive and\

                command in ["create", "delete", "delete_all", "add_initiator",

                            "allow_any_host", "split_bdev", "add_lun",

                            "add_pg_ig_maps", "remove_target", "add_secret",

                            "destruct_split_bdev", "delete_pmem_pool",