aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* [transport] Prevent package list for unresponsive nodesTrevor Benson2024-01-021-0/+2
| | | | | | Raise error when _connect() returns non zero status. Signed-off-by: Trevor Benson <trevor.benson@gmail.com>
* [packit] Configure copr builds for centos streamilluminatus2023-12-291-0/+2
| | | | Signed-off-by: Trevor Benson <trevor.benson@gmail.com>
* [openvswitch] Add new get-mempool-info and driverctl cmdsKevin Traynor2023-12-261-1/+4
| | | | | | | | | | | 'get-mempool-info' without specifying a netdev will capture all the dpdk mempools that exist including ones no longer associated with a netdev but yet to be freed. 'driverctl list-persisted' will list the overrides that are persistent through reboots. Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
* [lustre] Collect most paramsNathaniel Clark2023-12-261-18/+18
| | | | | | | | | | | | Collect almost all params, except those explicitly excluded. Those excluded are known to be less useful debugging, and/or known to be very large. Some params are still split out to help limit size. Some very large (but possibly useful) param are relegated to `--all-logs` Signed-off-by: Nathaniel Clark <nclark@whamcloud.com>
* [slurm] Add configless supportArif Ali2023-12-261-10/+20
| | | | | Closes: #3439 Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [cockpit] Expand plugin to capture cockpit-bridgeJose Castillo2023-12-261-3/+5
| | | | | | | | Expand the cockpit plugin to capture configuration and output of cockpit-bridge. Also remove deprecated command 'remotectl'. Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [omnipath] Enable UbuntuPluginArif Ali2023-12-262-4/+5
| | | | | | Also add `opa-fm` service check Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [arcconf] add getconfig and GETLOGS commands for all available controllersSourabh Jain2023-12-181-7/+32
| | | | | | | | | | | | | | | Currently, the arcconf getconfig and GETLOGS commands are added with the assumption that the system has only one controller with ID 1. It is possible that a system can have multiple controllers, so commands need to be added to extract information for all available controllers. The arcconf utility does not have a dedicated command to get the list of controllers available in the system. Therefore, parse the 'arcconf list' output to obtain the controller ID for all controllers in the system, and add getconfig and GETLOGS commands for each available controllers. Signed-off-by: Sourabh Jain <sourabhjain@linux.ibm.com> Suggested-by: Borislav Stoymirski <borislav.stoymirski@bg.ibm.com>
* [maas] fix working directory for rsyslog configs on snap installAdam Vest2023-12-151-1/+1
| | | | Signed-off-by: Adam Vest <adam.vest@canonical.com>
* [hardware] Include basic info from device treeSimon Glass2023-12-141-0/+2
| | | | | | | | ARM devices often don't have SMBIOS information, so dmidecode does not return anything useful. Add the device tree model information and compatible string, which at least provides the basics. Signed-off-by: Simon Glass <sjg@chromium.org>
* [nvidia] collect nvidia-persistenced service statusSourabh Jain2023-12-141-0/+1
| | | | | | | | | Having nvidia-persistenced service status helps debugging issues with NVIDIA kernel driver behaviour with persistence mode on. Signed-off-by: Sourabh Jain <sourabhjain@linux.ibm.com> suggested-by: Borislav Stoymirski <borislav.stoymirski@bg.ibm.com> Tested-by: Borislav Stoymirski <borislav.stoymirski@bg.ibm.com>
* [mellanox_firmware] Add Mellanox firmware pluginAlin-Gabriel Serdean2023-12-131-0/+141
| | | | | | | | | | | | | | | | | | | This patch reports the output of the following Mellanox firmware commands: mlxdump mstconfig mstdump mstflint mlxreg mlxlink mlxcables mst Additionally, if the user allows system changes, we will atempt to start the mst device, add the cables and stop the device in the end. Signed-off-by: Alin-Gabriel Serdean <aserdean@ovn.org>
* [doc] Update README.md with new way to generate html docsJose Castillo2023-12-131-1/+1
| | | | | | | | | | | Commit ac17ca4f4 removed the command build_sphinx from setup.py, but the README.md file still references it. This patch offers an alternative for uses wishing to generate html docs, using the command sphinx-build. Related: RH: RHEL-17924 Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [pam] Collect "authselect current"Pavel Moravec2023-12-131-0/+1
| | | | | | | | | Collect current auth profile details. Resolves: #3438 Relevant: RHEL-18576 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [discovery] collect also logs from container discovery-toolboxroot2023-12-131-5/+2
| | | | | | | | As part of the discovery is also discovery-toolbox plugin we should collect also logs from this plugin which may be helpful in debugging issues. Signed-off-by: Jan Jansky <jjansky@redhat.com>
* [foreman] Change scope of tasks historyPavel Moravec2023-12-071-7/+7
| | | | | | | | | | | Collect up to 14 days of dynflow data instead of one month. Also, change granularity of the plugin option from months to days - we are almost never interested in tasks older than one month. Resolves: #3437 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [report] Update preset opts already in SoSComponentPavel Moravec2023-12-052-23/+26
| | | | | | | | | | | As SoSComponent already works with --tmp-dir and it can be declared in a preset, we need to move whole options evaluation from SoSReport to SoSComponent constructor. Resolves: #3425 Closes: #3432 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [docs] Fix warning generated when running sphinxJose Castillo2023-12-051-1/+1
| | | | | | | | | | | | | When running sphinx-build to generate html documentation, we get the following warning: sos/report/plugins/__init__.py:docstring of sos.report.plugins.Plugin.collection_file:15: WARNING: Inline literal start-string without end-string. Related: RH: RHEL-17923 Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [zvm] Don't require 'vmcp' and 'cpint' kernel modulesRenaud Métrich2023-12-051-4/+1
| | | | | | | These modules are not present anymore, at least on RHEL8. The modules are not needed to execute 'vmcp' commands. Signed-off-by: Renaud Métrich <rmetrich@redhat.com>
* [report] Clean tmpdir when adding or deleting presetPavel Moravec2023-12-021-3/+5
| | | | | | | | | | | | | When adding or deleting preset (and also when detecting no plugin leaves enabled), sos.report.execute must cleanup its working dir. Since we don't use return value of the execute method, we can skip the return command. Resolves: #3430 Closes: #3431 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [lvm2] Capture PV headers and metadata with pvckJose Castillo2023-12-021-1/+24
| | | | | | | | Capture PV headers and metadata when a new option, lvm2.metadata , disabled by default, is specified as plugin option. Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [report] Clean up reference to obsolete ticket-number optionJose Castillo2023-12-021-1/+1
| | | | | | Clean up one old reference to an obsolete option. Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* Add support for OSP 18 Openstack EDPM deploymentRoberto Alfieri2023-11-287-11/+97
| | | | | | | | | | | | | | EDPM (External DataPlane Management) will be the new way to deploy compute nodes startin from RHOSP 18 which will replace TripleO/Director - Added openstack_edpm plugin - Added custom path for config and logs in frr/libvirt/logrotate/openstack_neutron/openstack_nova/ovn_host modules - Tagged libvirt/openstack_neutron/openstack_nova/ovn_host modules as `openstack_edpm` Signed-off-by: Roberto Alfieri <ralfieri@redhat.com>
* [ansible] Skip collecting redundant dirs and files from /etc/ansiblePavel Moravec2023-11-271-2/+6
| | | | | | | | | | Forbid collecting some files and dirs from /etc/ansible that are not interesting for any investigation but might potentially collect sensitive data. Resolves: #3423 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [deb][pkg] Update control for downstream buildArif Ali2023-11-264-3/+7
| | | | | | | | | | | | | | | | | * python3-setuptools is still required for building the package * python3-yaml is required for build for tests with the conformance_tests * Add the same ignores as dowstream build, so that we don't have to keep changing these. Once the avocado is packaged, we can take this out * Update image for the deb creation, so that it can be minimal * Fix lxd runtime, so that if the root device is not available, it will not look at the storage pools Downstream build machines are much more minimal compared to gce instances, so we require more dependencies to be added. Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [grafana] Grab data from grafana snapArif Ali2023-11-261-11/+34
| | | | | | | | Closes: #3278 Co-authored-by: Pravesh Gaire <pravesh.gaire@canonical.com> Signed-off-by: Pravesh Gaire <pravesh.gaire@canonical.com> Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [networkmanager] Add config files to be collectedHiroyuki Yasuhara2023-11-221-1/+5
| | | | | | | Collect some additional configuration files based on the description of man 5 NetworkManager.conf Signed-off-by: Hiroyuki Yasuhara <h.yasuhara@fujitsu.com>
* [libvirt] Add snapshot xml filesArif Ali2023-11-221-0/+1
| | | | | Resolves: SET-383 Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [canonical-livepatch-onprem] Add CanonicalAdam Vest2023-11-191-0/+42
| | | | | | | | Livepatch on-prem server support Resolves: SET-382 Signed-off-by: Adam Vest <adam.vest@canonical.com>
* [deb][packaging] Use python3-packaging as defaultArif Ali2023-11-161-2/+2
| | | | | | | Related: #3398 Related: #3383 Resolves: SET-326 Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [juju][collect] Fix new juju collectionArif Ali2023-11-162-7/+58
| | | | | | | | | | | | New version of juju uses colorisation, and therefore juju status and json.loads doesn't load the juju status correctly. By using --no-color based on the version of juju this should fix this particular use-case Resolves: #3399 Resolves: SET-339 Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [MAAS] Remove snap TLS private key from reportJorge Merlino2023-11-141-1/+4
| | | | | | | Remove file /var/snap/maas/current/http/certs/regiond-proxy-key.pem so that it is not collected by sosreport Signed-off-by: Jorge Merlino <jorge.merlino@canonical.com>
* [debian] Packaging and testing for debian pkgsArif Ali2023-11-1215-9/+426
| | | | | | | | | | | | | | | | Initial start of building and testing the deb package This will improve the overall testing of sos over the Ubuntu releases, and also helps with the basis of thenadding further stagetwo testing in the future. This will help to pick up issues automatically rather than the manual testing. Add ubuntu-latest for deb CI build and testing. Update the GCE images for all ubuntu images for CI testing Co-authored-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com> Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com> Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [pacemaker] Add "pcs property config" commandYuki Inoguchi2023-11-071-1/+12
| | | | | | | | Due to the deprecation of the existing "pcs property list", this patch checks the version of pcs and executes "pcs property config" as an alternative." Signed-off-by: Yuki Inoguchi <inoguchi.yuki@fujitsu.com>
* [flatpak] Capture details about branchJose Castillo2023-11-071-4/+3
| | | | | | | | | Gather details about branch, so it complements the version number (which can be empty sometimes). Fixes: #3404 Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [package_manager] Remove unused method pkg_nvraPavel Moravec2023-11-071-14/+0
| | | | | | Resolves: #3408 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [collect] Refactor `get_pty` functionalityJake Hunsaker2023-11-076-41/+53
| | | | | | | | | | | | | | | | | | | The `get_pty` parameter for remote executed commands was both a bit of a misnomer and applied too broadly. Refactor this to `use_shell` to be more obvious about what the intent behind the option is, and default all transports to `False`, so that by default we do not wrap any commands in a bash shell. This may be overriden on a per-transport basis via the ned `_need_shell` property within transport subclasses. Further, this facility has been expanded to be allowed on a per-command basis from `SoSNode.run_command()` and wherever that is linked. Related: #3399 Related: #3400 Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
* [kernelrt] Update options format for tunaJose Castillo2023-11-061-2/+8
| | | | | | | | | | | In newer versions of tuna, the option -P has been substituted for 'show_threads'. Test if the new option is present in 'tuna --help' and if so, use it. Closes: RH SUPDEV-150 Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [lxd_runtime] Fix image metadataArif Ali2023-10-311-3/+4
| | | | | | | Sometimes the update_sources attribute is not available so cannot build the list Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [salt] gather more data, e.g. services, grains, pillars, and moreMarek Czernek2023-10-302-4/+64
| | | | Signed-off-by: Marek Czernek <marek.czernek@suse.com>
* [build] Actually indicate this requires packagingMiro Hrončok2023-10-283-2/+8
| | | | | | | | This is a followup for 56904018045fdbdad666e55f0a850bc365809e50 I hereby declare that you can do whatever you want with this commit: Signed-off-by: Miro Hrončok <miro@hroncok.cz>
* [nfs] remove duplicate rpcinfo_-p_localhost logYuki Inoguchi2023-10-281-1/+0
| | | | | | | | | | | | | | | | | This commit removes the duplicate "rpcinfo_-p_localhost" log from the sos_commands/nfs directory. Currently, the same log file is being createdin sos_commands/sunrpc/ directory, as shown below: # ls sos_commands/nfs/rpcinfo_-p_localhost sos_commands/nfs/rpcinfo_-p_localhost # ls sos_commands/sunrpc/rpcinfo_-p_localhost sos_commands/sunrpc/rpcinfo_-p_localhost Since rpcinfo command is included in rpcbind package, let's remove the one in the nfs directory. Signed-off-by: Yuki Inoguchi <inoguchi.yuki@fujitsu.com>
* [utilities|collect] Capture ImportError exceptionsJose Castillo2023-10-272-2/+12
| | | | | | | Capture two ImportError exceptions when calling __import__() Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [logs] Add auth.log collection to the logs pluginNikhil Kshirsagar2023-10-271-0/+1
| | | | | | Resolves: #3393 Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com>
* [opensvc] Add relay config dumpArnaud Veron2023-10-251-0/+1
| | | | Signed-off-by: Arnaud Veron <arnaud.veron@opensvc.com>
* [insights] collect recursive listing of three dirsPavel Moravec2023-10-251-0/+3
| | | | | | | Resolves: #3391 Closes: #3390 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [openvswitch] Add driverctl verbose optionKevin Traynor2023-10-201-0/+2
| | | | | | | | | | | | | | | | | | | | | | | driverctl commands give info about which drivers are used for devices. They list PCI address and driver used e.g. 0000:01:00.0 vfio-pci This can usually be correlated to the device type through lspci. However, lspci may not be available if only some plugins were captured as part of the report. Even when lspci info is available, it is still convienent to have the device type co-located with the driver and override information. Add verbose option to driverctl commands that support it to also list device type. e.g. (wrapped for line length) 0000:01:00.0 vfio-pci (82599ES 10-Gigabit SFI/SFP+ Network Connection (Ethernet 10G 4P X520/I350 rNDC)) Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
* [pulpcore] Scrub AUTH_LDAP_BIND_PASSWORD valuePavel Moravec2023-10-181-19/+8
| | | | | | | | Likewise in #3379, scrub the password also in pulpcore plugin. Resolves: #3389 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [flatpak] Add flatpak pkg managerJose Castillo2023-10-172-2/+37
| | | | | | Related: RH SUPDEV-136 Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [microshift] Fix microshift-etcd.scope journal retrievalPablo Acevedo Montserrat2023-10-161-1/+2
| | | | Signed-off-by: Pablo Acevedo Montserrat <pacevedo@redhat.com>