aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--sos/presets/redhat/__init__.py4
-rw-r--r--sos/report/plugins/jars.py8
-rw-r--r--sos/report/plugins/kdump.py4
-rw-r--r--sos/report/plugins/logs.py4
-rw-r--r--sos/report/plugins/mssql.py4
-rw-r--r--sos/report/plugins/networking.py8
-rw-r--r--sos/report/plugins/npm.py6
-rw-r--r--sos/report/plugins/ovirt.py4
-rw-r--r--sos/report/plugins/pacemaker.py14
-rw-r--r--sos/report/plugins/proxmox.py4
-rw-r--r--sos/report/plugins/sar.py8
-rw-r--r--sos/report/plugins/watchdog.py4
12 files changed, 36 insertions, 36 deletions
diff --git a/sos/presets/redhat/__init__.py b/sos/presets/redhat/__init__.py
index 1fd749da..9f6c44d5 100644
--- a/sos/presets/redhat/__init__.py
+++ b/sos/presets/redhat/__init__.py
@@ -29,7 +29,7 @@ RHOSP = "rhosp"
RHOSP_DESC = "Red Hat OpenStack Platform"
RHOSP_OPTS = SoSOptions(plugopts=[
'process.lsof=off',
- 'networking.ethtool_namespaces=False',
+ 'networking.ethtool-namespaces=False',
'networking.namespaces=200'])
RHOCP = "ocp"
@@ -40,7 +40,7 @@ RHOCP_OPTS = SoSOptions(
plugopts=[
'crio.timeout=600',
'networking.timeout=600',
- 'networking.ethtool_namespaces=False',
+ 'networking.ethtool-namespaces=False',
'networking.namespaces=200'
])
diff --git a/sos/report/plugins/jars.py b/sos/report/plugins/jars.py
index ca2c0803..c7604bb4 100644
--- a/sos/report/plugins/jars.py
+++ b/sos/report/plugins/jars.py
@@ -24,9 +24,9 @@ class Jars(Plugin, RedHatPlugin):
plugin_name = "jars"
profiles = ("java",)
option_list = [
- PluginOpt('append_locations', default="", val_type=str,
+ PluginOpt('append-locations', default="", val_type=str,
desc='colon-delimited list of additional JAR paths'),
- PluginOpt('all_known_locations', default=False,
+ PluginOpt('all-known-locations', default=False,
desc='scan all known paths')
]
@@ -51,11 +51,11 @@ class Jars(Plugin, RedHatPlugin):
jar_paths = []
locations = list(Jars.jar_locations)
- if self.get_option("all_known_locations"):
+ if self.get_option("all-known-locations"):
locations += list(Jars.extra_jar_locations)
# append also user-defined locations, if any
- user_locations = self.get_option("append_locations")
+ user_locations = self.get_option("append-locations")
if user_locations:
locations += user_locations.split(":")
diff --git a/sos/report/plugins/kdump.py b/sos/report/plugins/kdump.py
index 9ad71a5e..ea77a0c8 100644
--- a/sos/report/plugins/kdump.py
+++ b/sos/report/plugins/kdump.py
@@ -131,7 +131,7 @@ class AzureKDump(KDump, AzurePlugin):
packages = ('kexec-tools',)
option_list = [
- PluginOpt("get_vm_core", default=False, val_type=bool,
+ PluginOpt("get-vm-core", default=False, val_type=bool,
desc="collect vm core")
]
@@ -166,7 +166,7 @@ class AzureKDump(KDump, AzurePlugin):
self.add_copy_spec(f"{path}/*/kexec-dmesg.log")
# collect the latest vmcore created in the last 24hrs <= 2GB
- if self.get_option("get_vm_core"):
+ if self.get_option("get-vm-core"):
self.add_copy_spec(f"{path}/*/vmcore", sizelimit=2048, maxage=24)
# vim: set et ts=4 sw=4 :
diff --git a/sos/report/plugins/logs.py b/sos/report/plugins/logs.py
index 31448cc2..04352c7a 100644
--- a/sos/report/plugins/logs.py
+++ b/sos/report/plugins/logs.py
@@ -118,7 +118,7 @@ class IndependentLogs(LogsBase, IndependentPlugin):
class CosLogs(LogsBase, CosPlugin):
option_list = [
- PluginOpt(name="log_days", default=3,
+ PluginOpt(name="log-days", default=3,
desc="the number of days logs to collect")
]
@@ -127,7 +127,7 @@ class CosLogs(LogsBase, CosPlugin):
if self.get_option("all_logs"):
self.add_cmd_output("journalctl -o export")
else:
- days = self.get_option("log_days", 3)
+ days = self.get_option("log-days", 3)
self.add_journal(since=f"-{days}days")
# vim: set et ts=4 sw=4 :
diff --git a/sos/report/plugins/mssql.py b/sos/report/plugins/mssql.py
index 33582b91..592c094c 100644
--- a/sos/report/plugins/mssql.py
+++ b/sos/report/plugins/mssql.py
@@ -20,12 +20,12 @@ class MsSQL(Plugin, RedHatPlugin):
packages = ('mssql-server',)
option_list = [
- PluginOpt('mssql_conf', default='/var/opt/mssql/mssql.conf',
+ PluginOpt('mssql-conf', default='/var/opt/mssql/mssql.conf',
desc='SQL server configuration file')
]
def setup(self):
- mssql_conf = self.get_option('mssql_conf')
+ mssql_conf = self.get_option('mssql-conf')
# Pick error log file from mssql_conf.
# Expecting the following format
diff --git a/sos/report/plugins/networking.py b/sos/report/plugins/networking.py
index 9257dace..446ce29d 100644
--- a/sos/report/plugins/networking.py
+++ b/sos/report/plugins/networking.py
@@ -21,12 +21,12 @@ class Networking(Plugin):
option_list = [
PluginOpt("traceroute", default=False,
desc=f"collect a traceroute to {trace_host}"),
- PluginOpt("namespace_pattern", default="", val_type=str,
+ PluginOpt("namespace-pattern", default="", val_type=str,
desc=("Specific namespace names or patterns to collect, "
"whitespace delimited.")),
PluginOpt("namespaces", default=None, val_type=int,
desc="Number of namespaces to collect, 0 for unlimited"),
- PluginOpt("ethtool_namespaces", default=True,
+ PluginOpt("ethtool-namespaces", default=True,
desc=("Toggle if ethtool commands should be run for each "
"namespace")),
PluginOpt("eepromdump", default=False,
@@ -203,7 +203,7 @@ class Networking(Plugin):
self.add_cmd_output("ip netns")
cmd_prefix = "ip netns exec "
namespaces = self.get_network_namespaces(
- self.get_option("namespace_pattern"),
+ self.get_option("namespace-pattern"),
self.get_option("namespaces"))
if namespaces:
# 'ip netns exec <foo> iptables-save' must be guarded by nf_tables
@@ -254,7 +254,7 @@ class Networking(Plugin):
# Collect ethtool commands only when ethtool_namespaces
# is set to true.
- if self.get_option("ethtool_namespaces"):
+ if self.get_option("ethtool-namespaces"):
# Devices that exist in a namespace use less ethtool
# parameters. Run this per namespace.
self.add_device_cmd([
diff --git a/sos/report/plugins/npm.py b/sos/report/plugins/npm.py
index 5b373ab4..6753df24 100644
--- a/sos/report/plugins/npm.py
+++ b/sos/report/plugins/npm.py
@@ -18,7 +18,7 @@ class Npm(Plugin, IndependentPlugin):
plugin_name = 'npm'
profiles = ('system',)
option_list = [
- PluginOpt('project_path', default='', val_type=str,
+ PluginOpt('project-path', default='', val_type=str,
desc='Collect npm modules of project at this path')
]
@@ -35,9 +35,9 @@ class Npm(Plugin, IndependentPlugin):
)
def setup(self):
- if self.get_option("project_path"):
+ if self.get_option("project-path"):
project_path = os.path.abspath(os.path.expanduser(
- self.get_option("project_path")))
+ self.get_option("project-path")))
self._get_npm_output("npm ls --json", "npm_ls_project",
working_directory=project_path)
self._get_npm_output("npm config list -l",
diff --git a/sos/report/plugins/ovirt.py b/sos/report/plugins/ovirt.py
index 31a9dcde..d3eacc26 100644
--- a/sos/report/plugins/ovirt.py
+++ b/sos/report/plugins/ovirt.py
@@ -57,7 +57,7 @@ class Ovirt(Plugin, RedHatPlugin):
option_list = [
PluginOpt('jbosstrace', default=True,
desc='Enable oVirt Engine JBoss stack trace collection'),
- PluginOpt('sensitive_keys', default=DEFAULT_SENSITIVE_KEYS,
+ PluginOpt('sensitive-keys', default=DEFAULT_SENSITIVE_KEYS,
desc='Sensitive keys to be masked in post-processing'),
PluginOpt('heapdump', default=False,
desc='Collect heap dumps from /var/log/ovirt-engine/dump/')
@@ -231,7 +231,7 @@ class Ovirt(Plugin, RedHatPlugin):
sensitive_keys = self.DEFAULT_SENSITIVE_KEYS
# Handle --alloptions case which set this to True.
- keys_opt = self.get_option('sensitive_keys')
+ keys_opt = self.get_option('sensitive-keys')
if keys_opt and keys_opt is not True:
sensitive_keys = keys_opt
key_list = [x for x in sensitive_keys.split(':') if x]
diff --git a/sos/report/plugins/pacemaker.py b/sos/report/plugins/pacemaker.py
index 4575d333..faf3f6c3 100644
--- a/sos/report/plugins/pacemaker.py
+++ b/sos/report/plugins/pacemaker.py
@@ -25,9 +25,9 @@ class Pacemaker(Plugin):
)
option_list = [
- PluginOpt('crm_from', default='', val_type=str,
+ PluginOpt('crm-from', default='', val_type=str,
desc='specfiy the start time for crm_report'),
- PluginOpt('crm_scrub', default=True,
+ PluginOpt('crm-scrub', default=True,
desc='enable crm_report password scrubbing')
]
@@ -105,17 +105,17 @@ class Pacemaker(Plugin):
# time in order to collect data.
crm_from = (datetime.today() -
timedelta(hours=72)).strftime("%Y-%m-%d %H:%m:%S")
- if self.get_option("crm_from"):
+ if self.get_option("crm-from"):
if re.match(r'\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}',
- str(self.get_option("crm_from"))):
- crm_from = self.get_option("crm_from")
+ str(self.get_option("crm-from"))):
+ crm_from = self.get_option("crm-from")
else:
self._log_error(
- f"crm_from parameter '{self.get_option('crm_from')}' is "
+ f"crm_from parameter '{self.get_option('crm-from')}' is "
"not a valid date: using default")
crm_dest = self.get_cmd_output_path(name="crm_report", make=False)
- if self.get_option("crm_scrub"):
+ if self.get_option("crm-scrub"):
crm_scrub = '-p "passw.*"'
else:
crm_scrub = ""
diff --git a/sos/report/plugins/proxmox.py b/sos/report/plugins/proxmox.py
index 63db3199..42ff0018 100644
--- a/sos/report/plugins/proxmox.py
+++ b/sos/report/plugins/proxmox.py
@@ -26,7 +26,7 @@ class Proxmox(Plugin, DebianPlugin):
packages = ('proxmox-ve',)
option_list = [
- PluginOpt('output_formats',
+ PluginOpt('output-formats',
desc='List of output formats to use '
'for the commands separated by ":".',
default='text',
@@ -34,7 +34,7 @@ class Proxmox(Plugin, DebianPlugin):
]
def setup(self):
- output_formats = self.get_option('output_formats').split(':')
+ output_formats = self.get_option('output-formats').split(':')
commands = [
'cluster/resources',
diff --git a/sos/report/plugins/sar.py b/sos/report/plugins/sar.py
index 5721256f..c2dd7be7 100644
--- a/sos/report/plugins/sar.py
+++ b/sos/report/plugins/sar.py
@@ -25,7 +25,7 @@ class Sar(Plugin):
for the current day-of data being collected at the time of report
generation).
- Using the 'all_sar' plugin option will not only cause the plugin to capture
+ Using the 'all-sar' plugin option will not only cause the plugin to capture
_all_ 'saX' files present on the host, but further perform the 'sar'
conversion on all files, not just those produced within the last week.
@@ -46,13 +46,13 @@ class Sar(Plugin):
packages = ('sysstat',)
sa_path = '/var/log/sa'
option_list = [
- PluginOpt('all_sar', default=False,
+ PluginOpt('all-sar', default=False,
desc="gather all system activity records")
]
def setup(self):
self.add_copy_spec(self.path_join(self.sa_path, '*'),
- sizelimit=0 if self.get_option("all_sar") else None,
+ sizelimit=0 if self.get_option("all-sar") else None,
tailit=False)
try:
@@ -72,7 +72,7 @@ class Sar(Plugin):
sar_filename = 'sar' + fname[2:]
if sar_filename not in dir_list:
# only collect sar output for the last 7 days by default
- if not self.get_option('all_sar') and \
+ if not self.get_option('all-sar') and \
self.is_older_than_7days(sa_data_path):
continue
sar_cmd = f"sar -A -f {sa_data_path}"
diff --git a/sos/report/plugins/watchdog.py b/sos/report/plugins/watchdog.py
index 6ed3c8d8..27dda925 100644
--- a/sos/report/plugins/watchdog.py
+++ b/sos/report/plugins/watchdog.py
@@ -20,7 +20,7 @@ class Watchdog(Plugin, RedHatPlugin):
packages = ('watchdog',)
option_list = [
- PluginOpt('conf_file', default='/etc/watchdog.conf',
+ PluginOpt('conf-file', default='/etc/watchdog.conf',
desc='watchdog config file')
]
@@ -55,7 +55,7 @@ class Watchdog(Plugin, RedHatPlugin):
Collect configuration files, custom executables for test-binary
and repair-binary, and stdout/stderr logs.
"""
- conf_file = self.path_join(self.get_option('conf_file'))
+ conf_file = self.path_join(self.get_option('conf-file'))
log_dir = self.path_join('/var/log/watchdog')
# Get service configuration and sysconfig files