| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
Fix small typo in the chroot section of the man page (alwyas -> always)
Fixes #1035
Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
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 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>
|
|
|
|
| |
Signed-off-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
|
|
|
|
| |
Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
|
|
|
|
| |
Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
Resolve issue #886
Change-Id: I0496cf4f4dbee33a8a6c52b2b6a1a3271834fb6f
Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
/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>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
| |
Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
Acked-by: Flavio Leitner <fbl@redhat.com>
|
|
|
|
|
|
|
|
|
| |
"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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
test_exe.py fails with nosetests-3.
Updated to python 3 compatible syntax.
Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
Resolves: #980.
Signed-off-by: Sachin <psachin@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|