aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [ovirt] reduce line lengths by storing command names in localsBryn M. Reeves2018-03-281-10/+9
| | | | | | | The new oVirt command names are very long: define them once in local variables in setup() and reference these in collection calls. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ovirt] Capture domain information for v4 and resolve #947Jake Hunsaker2018-03-281-2/+22
| | | | | | | | | | | | | | | In ovirt 4.x, the engine-manage-domains tool is no longer used, being replaced by ovirt-aaa-jdbc-tool. This patch uses the correct utility based on which is available on the ovirt manager. Additionally, the jbosstrace option will now only work where ovirt-engine is installed (instead of just dwh), resolving #947. Resolves: #947 Closes: #1063 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] handle KeyboardInterrupt separately in execute()Bryn M. Reeves2018-03-281-2/+5
| | | | | | | Exit on user cancel should use 130 (128 + SIGINT) as the exit status. Treat it separately from other top-level exceptions. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] log exception detail when waiting on user inputBryn M. Reeves2018-03-281-2/+6
| | | | | | | | If we catch an exception while waiting on user input, exit with status 130 if the exception is KeyboardInterrupt, otherwise log the exception text and exit with the exception as the status. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] remove exception handling from SoSReport.cleanup()Bryn M. Reeves2018-03-281-11/+8
| | | | | | It's pointless catching exceptions only to re-raise them. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] Fix exit status propagationJake Hunsaker2018-03-282-3/+8
| | | | | | | | | | | | | | | This fixes exit status propagation so that sos will now properly exit with a non-zero exit code when appropriate, such as when a keyboard interrupt is issued. Additionally, this fixes an issue when a NameError would be hit when using --debug due to FileNotFoundError not existing in python 2.7. Resolves: #672 Closes: #1062 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [xfs] Remove logprint and add xfs statsJake Hunsaker2018-03-281-7/+1
| | | | | | | | | | | | | Removes the logprint option which wasn't particularly useful for sos to capture, reference Red Hat Bugzilla 1250035. Adds collection of /prox/fs/xfs which has useful xfs statistics. Resolves: #621 Closes: #1061 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] Replace optparse with argparseJake Hunsaker2018-03-271-125/+107
| | | | | | | | Optparse is deprecated, this patch converts sos to use argparse instead. Resolves #738 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
* [lustre] Add Lustre filesystem pluginNathaniel Clark2018-03-271-0/+57
| | | | | | | | | | | | | | | | This collects some basic information about the configuration of Lustre on either a client or a server, including: * Interal ring buffer log * Basic configuration information * Quota information * Configuration of LNet * JobID info * LDLM connection states Fixes: #1058 Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [nvme] Create nvme pluginRodrigo R. Galvão2018-03-271-0/+43
| | | | | | | | | | | | | This plugin collects configuration and system information about NVMe devices. The nvme-cli tool is used to get relevant information about firmware, supported commands, serial, model, usage, etc.. Fixes: #1053. Signed-off-by: Rodrigo R. Galvao <rosattig@br.ibm.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [scsi] scsi address mapping with scsi device nameRanjithML2018-03-271-1/+1
| | | | | | | | | | | | Collect scsi device name mapping with scsi "H C T L" address. Currently sg_map collect mapping information about sg device name and scsi address. "sg_map -x" collect scsi device name mapping with sg device name and scsi address. Fixes: #1014 Signed-off-by: RanjithML ranjithml@gmail.com Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [process] Replace stime option to lstart optionKeigo Noha2018-03-271-1/+1
| | | | | | | | | | | | stime option doesn't provide consistent formatted data and the granularity of time scale differs from time to date order. lstart option provides consistent date/time format and it is easy to find out when the process starts. Fixes: #1004 Signed-off-by: Keigo Noha <knoha@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [docker] Enable log timestamp in docker logsTakayoshi Kimura2018-03-131-1/+1
| | | | | | | | | | | Enable docker logs -t option to make container troubleshooting easier. Container logs sometimes have different TZ timestapms without TZ info or no timestamps at all. Fixes: #1013 Signed-off-by: Takayoshi Kimura <tkimura@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [runc] Add new pluginJake Hunsaker2018-03-131-0/+42
| | | | | | | | | | | | Adds a new plugin for the runC container runtime. Collects a list of containers runc is aware of and the collects ps, event and state output for each container. Fixes: #993 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [filesys] Add reporting of ext fragmentation with e2freefragAdrien Mahieux2018-03-131-3/+8
| | | | | | | Fixes: #992 Signed-off-by: Adrien Mahieux <adrien@mahieux.net> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ansible] new plugin: collect ansible config and do a basic ping checkHarald Klein2018-03-131-0/+38
| | | | | | | | | Assisted by: Kyle Squizzato <ksquizza@redhat.com> Fixes: #931 Signed-off-by: Harald Klein <hari@vt100.at> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [satellite] increase timeout of spacewalk-debug to 15minutesMarcelo Moreira de Mello2018-03-131-1/+2
| | | | | | | | | spacewalk-debug usually runs longer than the default 5 minutes timeout. Fixes: #989 Signed-off-by: Marcelo Moreira de Mello <mmello@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [memory] collect swapon verbose summaryBryn M. Reeves2018-03-131-1/+2
| | | | | | Fixes: #863 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [docs] clarify test requirements in README.mdBryn M. Reeves2018-03-131-1/+3
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [build] Added dependencies in requirementsSachin2018-03-133-1/+19
| | | | | | | | | * Updated instructions in README to install Python dependencies * Added an example to create HTML docs * Added instruction to optionally run 'make test' before sending a PR * Relavant changes made in .travis.yml to test requirements Signed-off-by: Sachin Patil <sacpatil@redhat.com>
* [sosreport] exit status propagation is brokenBryn M. Reeves2018-03-131-14/+20
| | | | | | Fixes: #745 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [process] Add thread centered output for psAlexandru Juncu2018-03-131-0/+1
| | | | | | | | | | | | | | Add out put for the following command: ps -elfL This is a more friendly output for viewing thread related information. Fixes: #668 Closes: #669 Signed-off-by: Alexandru Juncu <alexj@linux.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [iprconfig] style fixesBryn M. Reeves2018-03-131-5/+6
| | | | | | | | Fix format value style and add VIM mode tags. Fixes: #575 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [iprconfig] collect more informationJakub Čajka2018-03-131-0/+7
| | | | Signed-off-by: Jakub Čajka <jcajka@redhat.com>
* [openstack_ironic] collect introspection data for all nodesDmitry Tantsur2018-03-091-0/+16
| | | | Signed-off-by: Dmitry Tantsur <divius.inside@gmail.com>
* [openstack_ironic] collect PXE environment informationDmitry Tantsur2018-03-091-4/+3
| | | | | | | This adds two directories, /httpboot and /tftpboot, as well as version of iPXE boot images package. Signed-off-by: Dmitry Tantsur <divius.inside@gmail.com>
* [openstack_ironic] collect information about ironic-inspectorDmitry Tantsur2018-03-091-1/+23
| | | | | | | ironic-discoverd was renamed to ironic-inspector in the Liberty release. This change adds support for the new name. Signed-off-by: Dmitry Tantsur <divius.inside@gmail.com>
* [openstack_ironic] collect drivers, ports and port groupsDmitry Tantsur2018-03-091-1/+3
| | | | | | Existing collection of ports is extended with the --long option. Signed-off-by: Dmitry Tantsur <divius.inside@gmail.com>
* [policy] add optional 'release' value for packagesBryn M. Reeves2018-02-202-2/+8
| | | | | | | | | | | | | | | | | | An a new optional field to the data gathered for installed packages to include the 'release' string provided by the platform package manager, and implement a change to the redhat policy family to add this to the RPM query command for those distributions. Policies wishing to implement this just need to ensure that there is an additional '|' separated field in the data returned by the specified query command. If the additional field is present it is assumed to contain a release value and is stored in the package dictionary for the 'release' key. For policies that do not define the additional field in query commands the 'release' key will always be set to None. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [etcd] Do not collect private etcd keysJake Hunsaker2018-02-121-0/+1
| | | | | | | Prevents sos from capturing the /etc/etcd/ca directory and its contents, which is primarily private keys and the like. Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
* [opendaylight] Enable plugin by puppet-opendaylight packagePavel Moravec2018-02-121-1/+1
| | | | | | | | Required for ODL running in a container. Resolves: #1207 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [ipa] fix implicit concatenation of one copy_specPavel Moravec2018-02-121-1/+1
| | | | | | | | | Missing comma between "/var/log/pki/pki-ca-spawn.*" and "/var/log/pki/pki-tomcat/kra/debug" Resolves: #1195 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [rabbitmq] Log collection fixes when rabbitmq runs in a containerMichele Baldessari2018-02-121-3/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When rabbitmq is running inside a container all the rabbitmqctl commands will fail with some obscure (welsh?) error messages like: $ rabbitmqctl cluster_status erno" ienrirto rt elromgigneart ipnrge sienn td)o _booetr"r,o{r: b<a0d.ar2g.,0[>{ l_prim_loader,check_file_result,3,[]},{init,get_boot,1,[]},{init,get_boot,2,[]},{init,do_boot,3,[]}]}} init terminating in do_boot () When run inside containers we need to run any rabbitmqctl command inside the container, so that output is properly collected: $ docker exec -t rabbitmq-bundle-docker-0 rabbitmqctl cluster_status Cluster status of node 'rabbit@controller-0' ... [{nodes,[{disc,['rabbit@controller-0','rabbit@controller-1', 'rabbit@controller-2']}]}, {running_nodes,['rabbit@controller-2','rabbit@controller-1', 'rabbit@controller-0']}, {cluster_name,<<"rabbit@controller-0.localdomain">>}, {partitions,[]}, {alarms,[{'rabbit@controller-2',[]}, {'rabbit@controller-1',[]}, {'rabbit@controller-0',[]}]}] While we're at it we also collect the logs of any rabbitmq container. This is particularly useful in containerized openstack deployments where short-lived containers are used to set up the cloud. Since the info output by 'rabbitmqctl cluster_status' and 'rabbitmqctl list_policies' is already contained in 'rabbitmqctl report', we just remove the former two commands. First reported via RHBZ#1525620 Signed-off-by: Michele Baldessari <michele@acksyn.org> Signed-off-by: John Eckersberg <jeckersb@redhat.com>
* [openvswitch] Add DPDK port mempool information.Kevin Traynor2018-02-071-0/+12
| | | | | | | | | | "ovs-appctl netdev-dpdk/get-mempool-info <port>" will dump the mempool information for DPDK ports. This can be used to help debug memory issues with DPDK ports. Signed-off-by: Kevin Traynor <ktraynor@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ipa] use correct PKI directories for tomcat versionThorsten Scherf2018-01-171-5/+18
| | | | | | | | | | The PKI subsystem uses different folders in IPA v3 and v4 for the NSS DB and the configuration files. The plugin needs to take this into account. Closes: #1163 Signed-off-by: Thorsten Scherf <tscherf@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ipa] add apache profileMartin Basti2018-01-171-1/+1
| | | | | | | | | httpd error_log collected by apache plugin contains useful information about IPA API operations Closes: #1010 Signed-off-by: Martin Basti <mbasti@redhat.com>
* [ipa] add missing server install logsMartin Basti2018-01-171-1/+4
| | | | | | | | Few install logs have been missing in report. Closes: #1010 Signed-off-by: Martin Basti <mbasti@redhat.com>
* [ipa] add KRA logsMartin Basti2018-01-171-0/+4
| | | | | | | | | IPA v4 can be installed with KRA subsystem. Adding particular logs to plugin. Closes: #1010 Signed-off-by: Martin Basti <mbasti@redhat.com>
* [nis] enable plugin also when package or config file presentPavel Moravec2018-01-161-1/+3
| | | | | | | | | Enable nis plugin also when ypserv package is installed or if /etc/ypserv.conf exists. Resolves: #1180 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [autofs] service status shouldnt rely on init.dPavel Moravec2018-01-161-1/+1
| | | | | | | | | Replace init.d call by more generic "service" command applicable also to systemd. Resolves: #1183 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [rpm] Change sort to use -V to use natural version orderingPablo Iranzo Gómez2018-01-161-1/+1
| | | | | | | Closes: #1184, #1185 Signed-off-by: Pablo Iranzo Gómez <Pablo.Iranzo@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [pcp] really apply sizelimit to logs collectedPavel Moravec2018-01-111-1/+1
| | | | | | | | add_copy_spec must be called to files and not dirs, to apply sizelimit Resolves: #1187 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [postgresql] Call SCL pg_dump with proper pathPavel Moravec2017-12-081-35/+8
| | | | | | | | | Also stop storing pg_dump in an auxiliary tempdir but under regular sos_commands/postgresql directory. Resolves: #1154 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [plugins] set proper PATH for SCL commandsPavel Moravec2017-12-083-7/+35
| | | | | | | | | | | | As SCL packages are deployed under /opt/${provider}/${scl}/, calling a SCL command needs that prefix in any path in PATH. Consequently, distro-specific SCL default path prefix of the provider must be defined in sos policies. Relevant to: #1154 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [haproxy] catch exception when parsing haproxy.cfgPavel Moravec2017-12-081-5/+9
| | | | | | | | catch exception when parsed haproxy.cfg file isnt accessible Resolves: #1160 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [etcd] dont traceback when etcd package isnt installedPavel Moravec2017-12-051-6/+11
| | | | | | | | | catch exception when etcd package isnt installed and we inspect its version Resolves: #1159 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [travis] Update travis with sudo and new pythonBryan Quigley2017-11-291-5/+12
| | | | | | | | | | | | Bumps requirement for CI to work on Python 3.5, 3.6, 3.7-dev. To enable sudo we bump the release to trusty. Dropped the depth to 5 as we aren't doing git commands. Running --help and -l at the moment (but that should have found issue 938) Also running with --batch to test basic running. Signed-off-by: Bryan Quigley <bryan.quigley@canonical.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [origin] fix typo in oc adm diagnostics --prevent-modificationPavel Moravec2017-11-291-2/+2
| | | | | | | | The argument is singular --prevent-modification . Resolves: #1150 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [openstack_keystone] Properly collect (non)default keystone domainsPavel Moravec2017-11-291-11/+10
| | | | | | | | | | | - call crudini instead of (wrapper and not necessarily installed) openstack-config - collect default /etc/keystone/domains only when crudini fails - scrub passwords in /etc/keystone/domains as well, when collected Resolves: #1147 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [rear] Add new pluginJake Hunsaker2017-11-151-0/+54
| | | | | | | | | | | This adds a plugin for Relax and Recover, or rear, which is a backup and recovery tool. Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Fixes: #1121 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>