| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
Resolves: #1523
Signed-off-by: Yuan-Chen Cheng <yc.cheng@canonical.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a 'cantboot' preset to enable a small number of plugins that may be
useful when troubleshooting a situation in which normal system startup
is failing.
Longer term, it would be beneficial if this preset could be
automatically loaded if sos detects that it is running in a system that
has been booted to rescue/emergency mode.
Resolves: #1522
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Changes the 'pcs status' collection to display full status results.
Resolves: #1521
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Adds a new plugin to collect information about Cockpit, a browser-based
system configuration and management tool.
Resolves: #1491
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Adds collection of a pmstat summary output using 5 minute intervals for
the current day, and makes a root symlink to it in the archive.
Resolves: #1520
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These changes are only on the Debian/Ubuntu bit of logs
Persistent systemd journaling (/var/log/journal exists)
use that instead of collecting any kern/syslog/auth from /var/log.
We also implement a log_days similar to the RH bit, but we default
to 7 days instead of 3.
--all-logs removes the day limit and gets it all
Rsyslog style (assumed if /var/log/journal doesn't exist
default run: collect all 4 of syslog[.1] and kern.log[.1]
(Same as previous behavior plus auth.log).
--all-logs: collect syslog*, kern.log*, and auth.log*
The notable change for --all-logs is it used to just collect all of
/var/log which causes MANY issues.
Before this change using --all-logs on systems with persistent journal
we would collect duplicate log files as such:
* Collect all of syslog*/kern*/auth*
* Collect the entire journal via /var/log/journal
* Collect the last two boots of journal via running journalctl command
Even with this change we still collect the journal last/current boot
with whatever is copied by journal since log_days.
Resolves: #1519
Signed-off-by: Bryan Quigley <bryan.quigley@canonical.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Adds collection of port mappings for podman containers.
Resolves: #1518
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Adds collection of 'tuned-adm verify' to allow for quick and easy
reference as to if tunables have been changed on a system using tuned.
Resolves: #1513
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Adds collection of stonith fencing history and a list of configured
watchdogs.
Resolves: #1512
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Updates the vdo plugin to grab status information for each individual
volume rather than dumping everything into a single output. Additionally
collect vdostats output.
Resolves: #1510
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Now that RHEL 8 Beta is out, update the redhat policy dist_version to be
able to properly set the version.
Resolves: #1509
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
| |
Collect /sys/class/drm/*/edid .
Resolves: #1507
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Update the list of ethtool commands to include:
ethtool -e (EEPROM dump)
ethtool -P (permanent MAC address)
ethtool -l (channel/queue settings)
ethtool --phy-statistics
ethtool --show-priv-flags
ethtool --show-eee
All of the above are helpful in understanding the state of modern NICs.
And -P is nice to have as otherwise there is no reliable way to see the
permanent MAC of team ports.
Resolves: #1503
Signed-off-by: Patrick Talbert <ptalbert@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Adds a new plugin for the storage event monitoring daemon peripety.
Resolves: #1502
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In containerized OSP env we now do not have no mysql packages
installed outside the containers. Therefore the mysql plugin does
not auto run.
This adds the openstack-selinux package to the RH package list of
the mysql plugin like on the OSP plugins.
Resolves: #1501
Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Adds a new plugin for the stratis storage manager
Resolves: #1495
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The 'describe' option logic was incorrectly nested behind the 'all'
option logic - so users couldn't successfully use the 'describe' option
unless they also specified 'all'.
Fixes the alignment so that 'describe' can be used properly and
separately.
Additionally fixes an issue where the syntax for getting resources for a
specified namespace only worked for pods, and otherwise generated an
error.
Resolves: #1494
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We nest ThreadPoolExecutors to run plugins to enable whole-plugin
timeouts, with the timeout being handled by the nested threadpool.
However, calling shutdown() on the nested thread is incorrect in the
event of a plugin timeout, as it only cancels pending jobs, not
currently running jobs that the timed-out plugin is.
Instead, call _threads.clear() to forcibly cancel the running job, thus
allowing the thread the return to the higher level threadpool and allow
other plugins to run via that thread.
This still allows data captured up to the point of the timeout to be
written to the sosreport.
Fixes: #1446
Resolves: #1492
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since the temporary directories have already been set up by the
time we process the --list-* handling (plugins, profiles, presets),
we still need to follow the _cleanup() path at exit.
To avoid a duplicate _cleanup() and resulting File Not Found
exception, raise SystemExit out of these paths, which takes us
through the normal exception handling in SoSReport.execute() and
is consistent with all other paths through the command.
Resolves: #1594.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
The configs member is now a tuple: we can't just slam it into a
add_copy_specs() list or a do_file_sub() call and have it work.
Split out the a_c_s() and pass the tuple alone (it looks less ugly
than having a cons inside the list), and loop over the elements in
postproc to handle the possible file locations.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Present in the /sys/class/ directory is a fourth fc directory
called fc_vports. Collect attribute info for ports in this
directory as they are useful in troubleshoting fabric config
issues.
Resolves: #1490
Signed-off-by: John Pittman <jpittman@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
| |
The scsi_host attributes can be very important when identifying
LLD/shost level issues. Add host value collection via udevadm.
Signed-off-by: John Pittman <jpittman@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
The queue values for block devices are very important for
performance, alignment, and I/O size issues. Add these values
by default.
Signed-off-by: John Pittman <jpittman@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
Resolves: #1426
Signed-off-by: Ashish Humbe <ahumbe@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds syspurpose output collection which is a new tool to assist with
subscription manager on RHEL systems.
Also adds rct output for default product certificates.
Resolves: #1489
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Direct Rules are an increasingly common way to control the firewall,
but current sos plugin output does not get these.
Update the plugin to grab firwall-cmd output about Direct Rules, and
also pick up the 'state' output and newer logging settings.
Resolves: #1486
Signed-off-by: Patrick Talbert <ptalbert@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
While running some tests using sosreport in a snap, I ran into the
following issue:
Creating compressed archive...
[archive:TarFileArchive] An error occurred compressing the archive:
unorderable types: int() >= str()
This is because the `xz_version` variable might be set to a list
containing one integer, but it was later compared to a list of strings,
raising an exception.
Resolves: #1485
Signed-off-by: Pierre Equoy <pierre.equoy@canonical.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Remove /sbin/ for service command so it runs on ubuntu.
Move initctl command to upstart plugin
Resolves: #1484
Signed-off-by: Bryan Quigley <bryan.quigley@canonical.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
The outer loop was also double-indented (my local version of
pycodestyle did not pick this up, nor did my eyes...).
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Collect further ipvsadm details and always do so with the -n numeric
option.
Resolves: #1483
Signed-off-by: Patrick Talbert <ptalbert@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
Re-format the indentation and structure of the innermost loop in
the failed deployments capture.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
The OpenStack undercloud can support > 1 stack. This patch will iterate
through all the possible stacks deployed on the undercloud.
Signed-off-by: Joe Talerico <rook@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
When attempting to add a preset when /var/lib/sos/presets does not
exist, the attempt will fail. Now, if the directory does not exist, sos
will attempt to create it.
Resolves: #1480
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The manpage for sosreport was previously missing documentation for the
new preset feature in 3.6.
This updates the manpage to include entries for add-preset, del-preset,
list-presets, and the --desc options.
Resolves: #1479
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Inspect smart card reader information using opensc command
pkcs11-tool and collect its configuration.
Resolves: #1478
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Conventional DEB package extract juju binary in /usr/bin/juju,
but SNAP extract juju binary in /snap/bin/juju.
Fixes: #1475
Resolves: #1476
Signed-off-by: Eric Desrochers eric.desrochers@canonical.com
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
Resolves: #1474
Signed-off-by: Juan-Luis de Sousa-Valadas <jdesousa@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
In case do_file_sub fails to apply a substitution on a non-existing file,
don't log it as an error log to stdout.
Resolves: #1471
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Capture additional datapath flows for the OVS
netdev datapath with: ovs-appctl dpctl/dump-flows
Resolves: #1468
Signed-off-by: Andreas Karis <akaris@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Red Hat distributions have not used the ipsec-tools and racoon packages
since RHEL5, and all of this information is collected in the OpenSwan
module via the same paths.
Similarly, Debian distributions have not provided this since wheezy, and
the same data is collected in OpenSwan.
Resolves: #1465
Signed-off-by: Robb Manes <rmanes@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When collecting IPSec data, currently nothing prevents the
collection of keys or the cert.db files. Ensure we don't collect
this information as it is private.
Thanks to Robert Bost (@bostrt) for discovering this and Matt Rogers
(@mrogers950) for double checking what we should ignore.
Resolves: #1463
Signed-off-by: Robb Manes <rmanes@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch removes an unnecessary comparison on package's __path__
attribute with an empty string in get_modules method.
When we pass an empty path to _find_plugins_from_list function it
returns None. The NoneType is not an iterable entity, so this
will leads to runtime error when we extend the plugins list with
None value.
Resolves: #1463
Signed-off-by: Sourabh Jain <sourabhjain@linux.ibm.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Use append mode when writing to plugin exception files: otherwise,
if multiple exceptions occur within one plugin (e.g. in separate
execution phases), the later ones will overwrite the earlier.
Resolves: #1461
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Add mode (e.g. to allow append) support to add_string() function.
Related: #1461
Signed-off-by: Filip Krska <fkrska@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Newer versions of ovirt-engine can produce a short and readable
diff from the "factory defaults".
Resolves: #1460
Signed-off-by: Dan Kenigsberg <danken@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
Resolves: #1459
Signed-off-by: David Vallee Delisle <dvd@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Dont rely on device names from "ip -o link" output where the names
contain also physical device name. Rather get the list of device names
from listing /sys/class/net/ directory.
Resolves: #1458
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When running `sosreport -l`, or the more specific --list-* options,
sos still creates a temp directory and log files, however we were
not cleaning those up - so over time we could have a multitude of
sos temp directories littering the filesystem.
Now always cleanup the temp directory after execution. Only
exception to this would be if we hit an OSError during execution
_and_ the debug option is specified.
Resolves: #1456
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|