commit e0c55c8aa9c3ebf8a0c46dc258fa7bf4121caf84
author Brad Bishop <bradleyb@fuzziesquirrel.com> Tue Oct 12 17:14:07 2021 -0400
committer Brad Bishop <bradleyb@fuzziesquirrel.com> Wed Oct 13 10:40:11 2021 +0000
tree 488659a928f69fec10dbce2354d8e08d37132530
parent 31a784471db3ee9c8f195ed27581baae0e2aa872

pldm_visualise_pdrs.py: make --user optional

Most ssh clients (ssh, scp, etc) will default to the currently logged in
user if unspecified elsewhere, including paramiko itself.

Make --user optional and default to the currently logged in user to be
consistant with the behavior of other ssh client applications.  This
also avoids the unusual sitation where --user is only required if it
does not appear in an ssh configuration file host entry (useful if
support for parsing ssh configurations were to be added in the future).

Change-Id: I0d4fdd51d03c205b1457ea199934fda2e3a2ab29
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

tools/visualize-pdr/pldm_visualise_pdrs.py

diff --git a/tools/visualize-pdr/pldm_visualise_pdrs.py b/tools/visualize-pdr/pldm_visualise_pdrs.py
index 1a9d087..f81cf23 100755
--- a/tools/visualize-pdr/pldm_visualise_pdrs.py
+++ b/tools/visualize-pdr/pldm_visualise_pdrs.py
@@ -197,8 +197,7 @@
     parser = argparse.ArgumentParser(prog='pldm_visualise_pdrs.py')
     parser.add_argument('--bmc', type=str, required=True,
                         help="BMC IPAddress/BMC Hostname")
-    parser.add_argument('--user', type=str, required=True,
-                        help="BMC username")
+    parser.add_argument('--user', type=str, help="BMC username")
     parser.add_argument('--password', type=str, required=True,
                         help="BMC Password")
     parser.add_argument('--port', type=int, help="BMC SSH port",