aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [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>
* [docker] fix copy&paste error in a for cyclePavel Moravec2017-11-151-1/+1
| | | | | | | | "containers" is an unknown variable, "insp" is the correct one Resolves: #1148 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [global] update FSF addressSandro Bonazzola2017-11-153-9/+9
| | | | | | | | | | | | | Update FSF address for the following plugins: - boom - conntrackd - sas3ircu Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com> Fixes: #1146 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [opendaylight] new plugin for OpenDaylightPavel Moravec2017-11-151-0/+38
| | | | | | | | Collecting logs and configs. Resolves: #1143 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [openvswitch] collect few ovs-ofctl dumps and OVS listPavel Moravec2017-11-151-0/+5
| | | | | | | | Required for/by OpenDaylight. Related to: #1143 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [cups] collect logs also via journalctlPavel Moravec2017-11-151-0/+2
| | | | | | | | | cups has options to log to logfiles or via journald, thus sosreport should collect both. Resolves: #1142 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [sas3ircu] add missing descriptionKazuhito Hagio2017-11-151-0/+2
| | | | Signed-off-by: Kazuhito Hagio <k-hagio@ab.jp.nec.com>
* [haproxy] Fix py2 specific import syntax for urlparseLouis Bouchard2017-11-151-1/+5
| | | | | | | | | | | | | urlparse is now part of urllib in python3. Make sure that the proxy behaves correctly on both versions. Closes: #1137 Signed-off-by: Louis Bouchard <louis@ubuntu.com> Fixes: #1138 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [vdo] collect proper sys pathsPavel Moravec2017-11-151-2/+2
| | | | | | | | | VDO plugin shall collect /sys/kvdo and /sys/uds instead of original /sys/vdo and /sys/albireo Resolves: #1134 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [process] inhibit lsof's thread information by defaultKazuhito Hagio2017-11-151-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | Current lsof lists not only processes' open file information but also all threads' one by default. But when a heavily-multithreaded process exists, this behavior takes several minutes and consumes huge CPU time. Moreover, if it takes more than 300 seconds, lsof executed by sosreport times out with no useful information. # ./pthread 12000 & # time lsof -b +M -n -l >/dev/null ... real 4m57.620s user 1m13.004s sys 3m44.350s This patch inhibits lsof's thread information with its -c option to reduce CPU time and adds an option to gather it if needed. Signed-off-by: Kazuhito Hagio <k-hagio@ab.jp.nec.com> Fixes: #1133 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [etcd] fix typo in etcdctl subcmdsPavel Moravec2017-11-151-1/+1
| | | | | | | | subcmds variable set while "subcmd" referred instead. Resolves: #1141 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [openstack_[glance|heat|cinder|nova]] limit command runMartin Schuppert2017-11-154-51/+129
| | | | | | | | | | | | | | | | | Collect "glance-manage db_version" and similar commands from the four plugins only if the relevant services or containers are running. Otherwise the commands get stuck and timeout. This is an enhancement to #1124 to check for containers + do the same for nova + cinder. Signed-off-by: Martin Schuppert mschuppe@redhat.com Edited to remove use of shell syntax. Fixes: #1139 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack_nova] added missing nova container configMartin Schuppert2017-11-071-1/+5
| | | | | | | | | | | | | | Tripleo Pike opinionated config+log paths to be collected for services, when running in containers. The nova configuration for the nova and placement container was included, but the nova configuration libvirt container was missing. Also the httpd configs for the nova contaier were added. This is a change to #1130 Signed-off-by: Martin Schuppert mschuppe@redhat.com
* [sos] bump release to 3.53.5Bryn M. Reeves2017-11-022-2/+5
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [haproxy] avoid exception if no provision_ip defined3.5beta1Bryn M. Reeves2017-11-011-0/+7
| | | | | | | Ensure that provision_ip is always defined, and do not attempt to collect data if no address was found in haproxy.cfg. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [system] add zlib to verify_packages listBryn M. Reeves2017-11-011-1/+1
| | | | | | | | Fixes: #1079 Suggested-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [nss] add verify_packages for 'nss.*'Bryn M. Reeves2017-11-011-0/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openssl] verify all packages matching 'openssl.*'Bryn M. Reeves2017-11-011-0/+1
| | | | | | Related: #1079. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [distupgrade] add missing packages listBryn M. Reeves2017-11-011-0/+7
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>