| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
If the relevant environment variables are set, then we collect some
useful information to debug landscape-server further
Signed-off-by: Arif Ali <arif.ali@canonical.com>
Signed-off-by: Mark Cunningham <mark.cunningham@canonical.com>
|
|
|
|
| |
Signed-off-by: Pablo Acevedo Montserrat <pacevedo@redhat.com>
|
|
|
|
|
|
|
|
| |
In a contrail environment, the container name could also start with vrouter,
this then ensures that the relevant details are also collected from any host
that has the containers running
Signed-off-by: Arif Ali <arif.ali@canonical.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add masakari and masakarimonitors plugin for openstack. This is
a ubuntu plugin at the moment. These are split into 2, as they
run on 2 seperate set of nodes.
* masakari is a control node service
* masakarimonitors runs on a nova hypervisor
Resolves: #3348
Signed-off-by: Arif Ali <arif.ali@canonical.com>
|
|
|
|
|
|
|
|
|
| |
This enables ovn sos report plugin to collect logs ovnkube-controller
container because ovn-kubernetes now provides option to run both
ovnkube-node and ovnkube-controller in same container with this
PR https://github.com/ovn-org/ovn-kubernetes/pull/3807.
Signed-off-by: Periyasamy Palanisamy <pepalani@redhat.com>
|
|
|
|
| |
Signed-off-by: Trevor Benson <trevor.benson@gmail.com>
|
|
|
|
|
|
|
|
|
|
| |
Since OVS 3.2 there is a command to capture the sleep
configuration of DPDK PMD cores.
This shows the maximum amount of time the PMD cores may
sleep for under low or no load traffic conditions.
Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In MAAS 3.5, system journal is used by all the services, therefore the
log size is significantly increased compared to the previous MAAS
versions. For snap this means that `journalctl` might need a long time
to collect and filter the log files before outputting them.
This sometimes leads to `maas` plugin timeouts with 3.5 snap, as `sos`
performs automatic log collection for services listed in `services`
tuple without any filters other than unit name, unless explicitly
specified by user. We want the plugin to work without additional params,
so we service declaration for 3.5 snap is ommitted. Logs are collected
via `add_journal` call with a sane default limit to avoid timeouts.
Signed-off-by: Igor Brovtsin <brigaccess@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
| |
The sosreport limits to collect logs at maximum of 25 MB in a given
collection passed into add_copy_spec method. so this may lead into
logs wouldn't have fully collected when user collected sos report
without --all-logs option.
Hence this commit ensures logs and dbs collected as much as possible
when --all-logs option is not specified.
Signed-off-by: Periyasamy Palanisamy <pepalani@redhat.com>
|
|
|
|
|
|
|
|
| |
With Interconnect support in latest OVN-Kubernetes, ovnkube-nodes
logs grew large. This commit adds the ability to collect those
additional logs.
Signed-off-by: Periyasamy Palanisamy <pepalani@redhat.com>
|
|
|
|
| |
Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
|
|
|
|
|
| |
Signed-off-by: Arif Ali <arif.ali@canonical.com>
Signed-off-by: Adam Vest <adam.vest@canonical.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change grep_for_content to grep for a fixed string by default.
The reason is tests would match 1a2b3c4 as IP address 1.2.3.4 (which it
is not).
Add regexp option to grep_for_content, to allow the default "grep" search.
Resolves: #3320
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
This PR adds the nat table to default_ip_tables
so when reading /proc/net/ip_tables_names fails,
it captures all the default tables.
Related: RHBZ#2228642
Signed-off-by: Jose Castillo <jcastillo@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since Python 3.12, archive.extractall would require setting an
extraction filter otherwise an exception is raised. See PEP 706 for
more.
Since Python 3.10 and 3.11 can raise false alarms as exceptions (see #3330
for examples), we must use the legacy fully_trusted filter.
Relevant: #3319
Closes: #3330
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
| |
Signed-off-by: Arif Ali <arif.ali@canonical.com>
|
|
|
|
| |
Signed-off-by: Arif Ali <arif.ali@canonical.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Two new alerts:
sos/report/plugins/release.py:9:52: E231 missing
whitespace after ','
sos/report/plugins/__init__.py:456:21: E721 do not
compare types, for exact checks use `is` / `is not`,
for instance checks use `isinstance()
Signed-off-by: Jose Castillo <jcastillo@redhat.com>
|
|
|
|
|
|
|
| |
Slurm is a workload manager in the HPC space, this is a start on this,
and there may be further additions in the future.
Signed-off-by: Arif Ali <arif.ali@canonical.com>
|
|
|
|
|
|
|
|
|
| |
Relevant: rhbz2187407
Resolves: #3328
Author: Nitin U. Yewale <nyewale@redhat.com>
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Adds a predicate to avoid loading the tls module if
it is not loaded.
Resolves: #3326
Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com>
|
|
|
|
|
|
|
|
|
| |
Enable the plugin by running containers and collect some further system
logs.
Resolves: #3324
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
| |
ceph_common is triggered due to librados2 package. Remove it.
Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com>
|
|
|
|
| |
Signed-off-by: Trevor Benson <trevor.benson@gmail.com>
|
|
|
|
|
| |
Signed-off-by: Nagoor Shaik <nshaik@redhat.com>
Signed-off-by: Nagoor Shaik <nagoor.s@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since OVS 3.1 'ovs-appctl dpif-netdev/pmd-rxq-show' also
supports showing the pmd rxq % usage over shorter timeframes
than the default 1 minute, down to a granularity of 5 secs.
A user testing may send traffic and have a steady state for
less than 1 min, so it is useful to also see pmd rxq % usage
over some shorter time periods.
Add pmd-rxq-show stats for 5 and 30 secs.
Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If sos is being used in a live environment to diagnose an issue, using
sysroot can cause the network device enumeration via /sys/class/net
crawling to fail. This will be the case for systems that do not use
`nmcli`.
When in a live environment, network devices will not be under
`/$sysroot/sys/class/net` but the "regular" path for the booted
environment. Similarly, if sos is being run in a container that is
properly configured, network devices will appear under `/sys/class/net`
and not (necessarily) under the sysroot path that mounts the host's
filesystem.
As such, disregard a configured sysroot when enumerating network devices
by crawling `/sys/class/net`, and trap any exceptions that may percolate
up from this in edge case environments.
Related: #3307
Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
|
|
|
|
|
|
|
| |
This updates openshift_ovn plugin to collect ovn db files
when it is running with interconnect configuration.
Signed-off-by: Periyasamy Palanisamy <pepalani@redhat.com>
|
|
|
|
|
|
| |
This commit marks the release of sos-4.5.6.
Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
|
|
|
|
|
|
|
|
| |
This prevents mb_groups from being collected as it is large, and not
useful for debug.
This also collects more state of the filesystem.
Signed-off-by: Nathaniel Clark <nclark@whamcloud.com>
|
|
|
|
| |
Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com>
|
|
|
|
|
|
|
|
| |
This lets files triggers use * and also updates the
ceph plugins to trigger only if folders are non empty
Closes: #3311
Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com>
|
|
|
|
|
|
|
|
|
| |
Addresses ConfigParser.readfp removal in 3.12.
Fixes #3308
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=2223526
Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
|
|
|
|
|
|
|
|
| |
Add a common regex for re.MULTILINE so that doesn't need to be defined
and remove the definitions where `(?m)` is being defined
Closes: #3301
Signed-off-by: Arif Ali <arif.ali@canonical.com>
|
|
|
|
|
|
| |
Collect the microceph data in the ceph plugins.
Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com>
|
|
|
|
|
|
|
| |
MAAS no longer uses apache2 for it web hosting, so no longer required
for this plugin
Signed-off-by: Arif Ali <arif.ali@canonical.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Adding workdlow to publish the released sos to the latest/candidate channel
when the a new release is created via the tag. This ensures less mannual
intervention for future releases.
Update the snap creation for anyone building manually, and need python3-venv
as a build dependancy.
Signed-off-by: Arif Ali <arif.ali@canonical.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some of mac addresses was not properly obfuscated because
some collected data contains mac addresses in format
01:<mac> and parser was not ready for that.
Also added mapper which will obfuscate mac address in case
it is in format with _ instead of : as for example
00_50_56_87_5d_01
instead of
00:50:56:87:5d:01
Format with _ is used for example by vmware plugin.
Resolves: #3302
Signed-off-by: Jan Jansky <jjansky@redhat.com>
|
|
|
|
| |
Signed-off-by: Pablo Acevedo Montserrat <pacevedo@redhat.com>
|
|
|
|
|
|
|
|
| |
Resolves issue SUPDEV-137.
Adds a plugin option for the ssh module, defining whether it will or will not collect .ssh config files per user
Default for new option is True
Signed-off-by: Daniel Zhou <dzhou@redhat.com>
|
|
|
|
|
|
|
|
| |
Add a test that cleaner keeps permissions of a sanitised file
Relevant to: #3292
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
When copying files we applied a substitution in, we must replace just
original file content (shutil.copyfile) and not also its stat data
(shutil.copy).
Resolves: #3292
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Update the maintainer field to be more generic and direct readers to the
upstream repo.
Closes: #3290
Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
sos report does not gather the symlink
/etc/grub2-efi.cfg when present.
If a system is legacy, but has this file,
grubby default to using this one first in
some cases. Therefore we need to know if
it's present or not.
Closes: RHBZ#2218563
Signed-off-by: Jose Castillo <jcastillo@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch helps simplify the capture of data
from ceph plugins. After the original code was
divided in 8 different plugins, it may be a bit
confusing which plugins to use when running
sos and attempting to capture only ceph data.
By selecting the profile, sos will activate only
these ceph_* plugins that apply to a specific
ceph node.
Signed-off-by: Jose Castillo <jcastillo@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
options_tests.py tests -n option which means tens of plugins are
redundantly collected. Testing -o option is effectively the same while
the sosreport run needs a half time only.
The functionality of -n option is tested elsewhere, thus the change does
not shrink test coverage.
Resolves: #3288
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
| |
This commit marks the release of sos-4.5.5.
Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
|
|
|
|
|
|
|
|
| |
Tested by:
- manually patching file on Debian 11 and 12 (both on sos version 4.0.2)
- sudo ./bin/sos report from this commit (and on python 3.11)
Signed-off-by: Johan Guldmyr <johan@meetiqm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When calling get_cmd_output_path() to create a directory
via the plugin hpssm, sos threw the following exception:
Traceback (most recent call last):
File "/root/sos/sos/report/__init__.py", line 1224, in setup
plug.setup()
File "/root/sos/sos/report/plugins/hpssm.py", line 67, in setup
logpath = self.get_cmd_output_path()
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/sos/sos/report/plugins/__init__.py", line 2168,
in get_cmd_output_path
os.makedirs(cmd_output_path)
File "<frozen os>", line 225, in makedirs
FileExistsError: [Errno 17] File exists:
'/var/tmp/sos.1gdy83zb/sosreport-localhost-vbwfnpn/sos_commands/hpssm'
This was happening because the directory 'hpssm' was already created.
With this change we avoid any race where we call os.makedirs() after
we have already created the plugin directory.
Closes: RHBZ #2216608
Signed-off-by: Jose Castillo <jcastillo@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Two conntrack commands require several kmods - guard collection of the
commands by relevant SoSPredicate.
Resolves: #3279
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|