aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [openstack plugins] run openstack command when correct ENV is set (#1048)stuggi2017-06-288-24/+56
| | | | | | | | | With Tripleo Pike the default ENV changed from OS_TENANT_NAME to OS_PROJECT_NAME. The commands of the openstack plugins against the env should run when either OSP_TENANT_NAME or OS_PROJECT_NAME is set. Signed-off-by: Martin Schuppert mschuppe@redhat.com Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [plugins] Tripleo specific containerized services logs (#1046)Bogdan Dobrelya2017-06-2819-35/+95
| | | | | | | | | | | | Add Tripleo Pike opinionated logs paths to be collected for services, when running in containers. This is a temporary and will be reworked for Queens, like switching those to syslog or fluentd shipping logs to Elasticsearch cluster. Partial bug: https://bugs.launchpad.net/tripleo/+bug/1700909 Related blueprint: https://review.openstack.org/#/c/462900 Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [targetcli] added one item (#1044)vredara2017-06-281-0/+1
| | | | | | collecting the /sys/kernel/config/target Signed-off-by: venkata edara <redara@redhat.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [gluster_block] Added new plugin gluster_block (#1041)vredara2017-06-281-0/+39
| | | | | | gluster_block plugin collects config/log files related to blocks Signed-off-by: venkata edara <redara@redhat.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [i18n] Update Spanish translation (#996)Adolfo Jayme-Barrientos2017-06-281-40/+34
| | | | Signed-off-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [iscsi] Enable plugin for Ubuntu and Debian (#932)Louis Bouchard2017-06-161-7/+17
| | | | Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [workflow] format update to PR template to list the items properlyAdam Stokes2017-06-141-4/+4
| | | | Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [workflow] Add a pull request template (#1025)Adam Stokes2017-06-141-0/+7
| | | | | | | | | | This includes information for users wishing to submit PR's and is visible during their creation of the PR. This allows them to validate that they are providing all necessary requirements prior to submitting and should reduce the amount of back and forth we do asking users to follow our guidelines. Fixes #1021 Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [AUTHORS] update authors file with more contributors (#1026)Adam Stokes2017-06-141-3/+103
| | | | | | | We've had a lot of contributions from outside companies, organizations, and individual contributors. I'd like to update this file to acknowledge those new contributors and continue to encourage and promote this products ecosystem. Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [hardware] add /sys/class/dmi/id/* (#1037)davecore822017-06-131-1/+2
| | | | | | | | | | | | | | | | | | | | | Collect the entire /sys/class/dmi/id/ directory from the hardware plugin. Today, this is really the "native" format for this information, as far as Linux is concerned. The dmidecode program (which historically at least read the data directly from firmware tables, as no proc or sysfs interface existed) is a legacy formatting of this information which we are unfortunately obliged to keep around a little longer yet, as many users have grown dependencies on its format. The data included in this directory is small (about 500 bytes on my system) and will not have an impact on the time or size of the report. Original discussion at: https://github.com/sosreport/sos/pull/1034 Signed-off-by: David Coronel <david.coronel@canonical.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [docker] Update Ubuntu copy spec (#1009)christyp2017-05-311-4/+3
| | | | | | | | The binary name changed from docker.io to docker. Update config and log file locations. Signed-off-by: Karsten Graul <kgraul@linux.vnet.ibm.com> Signed-off-by: Christy Perez <christy@linux.vnet.ibm.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* Fix dc-connect winbind logfile path for samba plugin (#1018)Günther Deschner2017-05-301-1/+1
| | | | | | Fixes patch collection on an incorrect samba path. Signed-off-by: Guenther Deschner <gd@samba.org> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [policies/redhat] make missing 'filesystem' package non-fatalBryn M. Reeves2017-05-221-2/+10
| | | | | | | | | | | | | | | | | | | | | | | On Red Hat systems, we use the version of the installed filesystem' package to enable handling of UsrMove[1] in the PATH used when running commands from sos. If the package is not present (e.g. because of a broken system configuration, rpm timeout, corrupt database or other cause), previous versions would fail immediately. In this situation there was no way for the user to attempt a complete run (short of fixing the problem related to the 'filesystem' package). Now that the timeout has been lengthened for this command, make the UsrMove determination a bit more robust to prevent users from being unable to run sos at all: - If RPM fails entirely, exit (plugin enablement tests will fail) - If RPM succeeds but 'filesystem' is missing assume UsrMove (this is true for all supported Red Hat distros today) - Otherwise apply the normal PATH logic. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ceph] exclude temporary mount locations from collectionTomas Petr2017-05-221-0/+7
| | | | | | | | | | Do not collect temporary mount paths at /var/lib/ceph/tmp/mnt.XXXX. Resolves #1006 Replaces #1005 Signed-off-by: Tomas Petr <tpetr@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack_neutron] Add more OpenStack Neutron commandsAssaf Muller2017-05-221-0/+4
| | | | | | | | | | | | | | | Finish off core set of Neutron resources: * Networks * Security groups * Floating IPs As well as the API extensions exposed by the server: * Extensions Resolves: #1011. Signed-off-by: Assaf Muller <amuller@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [global] update FSF addressSandro Bonazzola2017-05-22233-722/+743
| | | | | | | Resolve issue #886 Change-Id: I0496cf4f4dbee33a8a6c52b2b6a1a3271834fb6f Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
* [libvirt] fix per-process cgroup collectionBryn M. Reeves2017-05-151-1/+1
| | | | | | | The per-pid cgroup data is in a pseudofile named 'cgroup', and not 'cgroups' as in commit 2523ad5. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [kernel] add /sys/fs/pstore collection for all platforms (#1008)Chris Newcomer2017-05-122-2/+2
| | | | | | | | | | | | | | | | The pstore data should be collected upon sosreport run for all architecture (currently only enabled in powerpc.py) because the data there will include any past oops event and/or panic events. This data would assist diagnosing panics with little other data available. Also, the data included is a few kb in size, so it will not add much size to the output of the report. I just moved the /sys/fs/pstore collection from powerpc.py to kernel.py in order to have it collect for all architectures. Signed-off-by: Chris Newcomer <chris@thenewcomers.org> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [plugins] Handle stat errors on missing files (#991)Louis Bouchard2017-05-101-1/+4
| | | | | | | When the stat is done on a broken symlink, it will return either OSError or FileNotFoundError. Handle these and report accordingly. Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [networking] collect more link data (#988)qsn2017-05-101-1/+2
| | | | | | | Collect macsec configuration, and detailed link information from iproute. Signed-off-by: Sabrina Dubroca <sd@queasysnail.net> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [docker] Collect image inspect output (#995)Jake Hunsaker2017-05-101-4/+12
| | | | | | Adds output for 'docker inspect' for each unique image on the host. Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [snappy] new plugin to show snappy info (#999)BryanQuigley2017-05-101-0/+36
| | | | | | | | | | Adds a new plugin that lists information about snappy, including: * all installed snaps with version info (snap list --all) * Snappy's own version info (snap --version) * status info and logs for the snapd daemon * List snappy system changes (snap changes) Signed-off-by: Bryan Quigley <bryan.quigley@canonical.com> Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
* [pacemaker] Collect user-defined logfilePavel Moravec2017-05-091-0/+17
| | | | | | | | | | /etc/sysconfig/pacemaker or /etc/default/pacemaker can specify pacemaker's logfile that sos should collect. Resolves: #1002. Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [pacemaker] Collect /etc/default/pacemaker for Debian/UbuntuBryn M. Reeves2017-05-091-2/+12
| | | | | | | | | Make /etc/default/pacemaker the default location for the Pacemaker defaults file, and have the RedHatPlugin override this to the /etc/sysconfig/pacemaker location as required. Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [policies/redhat] accept 'oci' as a valid container typeBryn M. Reeves2017-05-031-1/+1
| | | | | | | | | | | Currently the Red Hat container policy accepts 'container=docker' as a valid indication that we are running in a container and to enable sysroot and host sysroot checks. More recent Red Hat container infrastructure has change to setting 'container=oci', so recognise this value as well. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugins] work around Six string problems in HTML reportsBryn M. Reeves2017-05-031-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A workaround for Six string encoding problems involving strings that end in '\' characters was introduced for plain text reports in commit 3d23564: a similar fix is also needed for HTML reports, since the same string encoding problem can occur there too: > /usr/lib/python2.7/site-packages/six.py(647)u() -> return unicode(s.replace(r'\\', r'\\\\'), "unicode_escape") (Pdb) bt /usr/sbin/sosreport(25)<module>() -> main(sys.argv[1:]) /usr/lib/python2.7/site-packages/sos/sosreport.py(1632)main() -> sos.execute() /usr/lib/python2.7/site-packages/sos/sosreport.py(1606)execute() -> self.html_report() /usr/lib/python2.7/site-packages/sos/sosreport.py(1373)html_report() -> self._html_report() /usr/lib/python2.7/site-packages/sos/sosreport.py(1434)_html_report() -> self.handle_exception() /usr/lib/python2.7/site-packages/sos/sosreport.py(1432)_html_report() -> html = plug.report() /usr/lib/python2.7/site-packages/sos/plugins/__init__.py(930)report() -> + "/" + _to_u(cmd['file']) /usr/lib/python2.7/site-packages/sos/plugins/__init__.py(44)_to_u() -> s = six.u(s) > /usr/lib/python2.7/site-packages/six.py(647)u() -> return unicode(s.replace(r'\\', r'\\\\'), "unicode_escape") Avoid this by applying the same workaround ('\$' -> '\ $') in the existing Plugin _to_u() helper function. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [reporting] imrove readability of Six string workaroundBryn M. Reeves2017-05-031-7/+18
| | | | | | | | | | | Improve the readability of the changes to work around Six issue emphasise the fact that the buffer contains line oriented data, and introduce a helper function to test whether the string needs the workaround to be applied or not: this shortens the list comprehension which would otherwise overflow a single line due to the longer buffer variable name. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sos] update generated __init__.pyBryn M. Reeves2017-05-021-1/+1
| | | | | | | | | | | For hysterical reasons (RPM builds), the version string stored in the module file for the sos module is substituted from a value stored in the top level Makefile: this needs to be regenerated and committed each time the version changes. Update this for 3.4. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack_instack] extended packages listMartin Schuppert2017-05-021-0/+9
| | | | | | | | | | Extended packages list to verify triplo packages if --verify is specified. Resolves: #1001. Signed-off-by: Martin Schuppert <mschuppe@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack_*] fix issue with --verify optionMartin Schuppert2017-05-0213-13/+13
| | | | | | | | | | If --verify option got specified, the package list was not handled properly and resulted in trace. Resolves: #1000. Signed-off-by: Martin Schuppert <mschuppe@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [virsh] Handle properly cases when virsh commands failPavel Moravec2017-04-231-7/+7
| | | | | | | | | When parsing output of "virsh -r *" commands, handle properly cases when either command fails - do not parse the output then. Resolves: #997 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [ceph] fix list formattingBryn M. Reeves2017-04-201-2/+3
| | | | | | | | Fix the formatting of the add_copy_spec() lists in ceph: make the first conform to the sos bracing and indenting rules for lists in argument tuples, and fix a missing comma in the second. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [Plugin] remove bogus checks from get_cmd_output_now()Bryn M. Reeves2017-04-191-4/+0
| | | | | | | | The underlying get_command_output() function already checks the return status of the command: checking it again in the _now() version is useless. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [Pugin] revert 77eb4ab (do not return output from failed commands)Bryn M. Reeves2017-04-191-3/+0
| | | | | | | | | | | Revert this commit: individual plugins must test the command status (if they care): plugins that attempt to access command output without testing command status are buggy and need to be fixed. Resolves: #986. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openvswitch] Add logs for ovsdb-server and ovs-vswitchd.Kevin Traynor2017-04-131-0/+2
| | | | | Signed-off-by: Kevin Traynor <ktraynor@redhat.com> Acked-by: Flavio Leitner <fbl@redhat.com>
* [openvswitch] Add additional captures to help debug with DPDK datapath.Kevin Traynor2017-04-131-1/+7
| | | | | | | | | "ovs-appctl dpctl/show -s" for DPDK datapath stats "ovs-appctl dpif-netdev/pmd-rxq-show" for port/queue to pmd core mapping "ovs-appctl dpif-netdev/pmd-stats-show" for pmd stats Signed-off-by: Kevin Traynor <ktraynor@redhat.com> Acked-by: Flavio Leitner <fbl@redhat.com>
* [jars] fix maven_id dictionary encodingBryn M. Reeves2017-03-312-3/+13
| | | | | | | | | | | | | | | | | | | The dictionary must contain string types, not byte arrays. Causes an exception when packing strings with Python3: [sos.sosreport:setup] executing 'sosreport -vvv --batch --debug -o jars' Setting up plugins ... Traceback (most recent call last): File "./sosreport", line 25, in <module> main(sys.argv[1:]) File "/home/breeves/src/git/sos/sos/sosreport.py", line 1632, in main sos.execute() TypeError: key b'version' is not a string Resolves: #984. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [tests] update test_exe.py for python3Sandro Bonazzola2017-03-311-1/+1
| | | | | | | test_exe.py fails with nosetests-3. Updated to python 3 compatible syntax. Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
* [sos] bump version to 3.43.4Bryn M. Reeves2017-03-281-1/+4
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [Plugin] do not return output from failed commandsBryn M. Reeves2017-03-281-0/+4
| | | | | | | If a command returns status of '1' in Plugin.get_cmd_output_now(), do not return output even if the command wrote to stdio. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ceph_ansible] new plugin: collect ceph-ansible site.yml and group_varsHarald Klein2017-03-281-0/+36
| | | | | | | | | Assisted by: Kyle Squizzato <ksquizza@redhat.com> Resolves: #929. Signed-off-by: Harald Klein <hari@vt100.at> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ceph] implement log limit and add additional commandsHarald Klein2017-03-281-11/+26
| | | | | | | | | | Resolves: #926. Signed-off-by: Harald Klein <hari@vt100.at> (edits for add_copy_spec() usage) Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [grafana] add missing package list and profile entriesSachin2017-03-281-1/+3
| | | | | | | Resolves: #980. Signed-off-by: Sachin <psachin@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack_ansible] Add OpenStack-Ansible pluginMajor Hayden2017-03-281-0/+47
| | | | | | | | | | This patch adds a plugin to retrieve configuration content, Ansible inventory, and cached facts from OpenStack-Ansible deployment nodes. Resolves: #922. Signed-off-by: Major Hayden <major@mhtx.net> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [virsh] Collect host, network, nwfilter, pool and more domain infoJake Hunsaker2017-03-281-5/+31
| | | | | | | | | | | | | | | This patch adds collection of host information such as memory and capabilities. Additionally, user-defined elements such as networks, nwfilters and storage pools are now collected. A listing of all elements is taken and then, if they exist, the element's xml is collected as well. Further, more domain (vm) information is collected - such as dominfo and domblklist for each domain reported by virsh. Resolves: #918. Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
* [storageconsole] convert add_copy_spec_limit -> add_copy_specBryn M. Reeves2017-03-281-13/+10
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [salt*] convert add_copy_spec_limit -> add_copy_specBryn M. Reeves2017-03-282-13/+6
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [grafana] convert add_copy_spec_limit -> add_copy_specBryn M. Reeves2017-03-281-3/+5
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack_*] convert add_copy_spec_limit -> add_copy_specBryn M. Reeves2017-03-282-12/+12
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [gnocchi] convert add_copy_spec_limit -> add_copy_specBryn M. Reeves2017-03-281-4/+4
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>