Add support to log collecter CLI options for port
Changes:
- Added CLI option for HTTPS and IPMI
Tested:
Tested on sanndbox script.
Change-Id: Idfa5679f92f03198505abfb8604d937bea245ef9
Signed-off-by: George Keishing <gkeishin@in.ibm.com>
diff --git a/ffdc/collect_ffdc.py b/ffdc/collect_ffdc.py
index ef96c6b..698b87b 100644
--- a/ffdc/collect_ffdc.py
+++ b/ffdc/collect_ffdc.py
@@ -27,6 +27,12 @@
@click.option("-u", "--username", help="Username of the remote host.")
@click.option("-p", "--password", help="Password of the remote host.")
@click.option(
+ "-port_https", default=443, show_default=True, help="HTTPS port value."
+)
+@click.option(
+ "-port_ipmi", default=623, show_default=True, help="IPMI port value."
+)
+@click.option(
"-c",
"--config",
default=abs_path + "/ffdc_config.yaml",
@@ -77,6 +83,8 @@
remote,
username,
password,
+ port_https,
+ port_ipmi,
config,
location,
type,
@@ -93,11 +101,15 @@
"\n********** FFDC (First Failure Data Collection) Starts **********"
)
- if input_options_ok(remote, username, password, config, type):
+ if input_options_ok(
+ remote, username, password, port_https, port_ipmi, config, type
+ ):
this_ffdc = ffdc_collector(
remote,
username,
password,
+ port_https,
+ port_ipmi,
config,
location,
type,
@@ -127,7 +139,9 @@
click.echo("\n********** FFDC Finishes **********\n\n")
-def input_options_ok(remote, username, password, config, type):
+def input_options_ok(
+ remote, username, password, port_https, port_ipmi, config, type
+):
r"""
Verify script options exist via CLI options or environment variables.
"""