aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [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>
* Prefer packaging.version over pkg_resourcesMiro Hrončok2023-10-161-2/+2
| | | | | | | | | | pkg_resources bundle and call packaging.version internally anyway. In RHEL 10 and Ubuntu, we have packaging in BaseOS/main. I hereby declare that you can do whatever you want with this commit: Signed-off-by: Miro Hrončok <miro@hroncok.cz>
* Instead of a parse_version -> version_parse function, use parse_versionMiro Hrončok2023-10-161-8/+28
| | | | | | | | | | | This seems a bit more straightforward and achieves the same result. This also defines the API of sos.utilities to avoid flake8 "imported but unused" I hereby declare that you can do whatever you want with this commit: Signed-off-by: Miro Hrončok <miro@hroncok.cz>
* Expect ImportError instead of SyntaxError when importing pkg_resourcesMiro Hrončok2023-10-161-1/+1
| | | | | | | | Getting SyntaxError from pkg_resources is unlikely. I hereby declare that you can do whatever you want with this commit: Signed-off-by: Miro Hrončok <miro@hroncok.cz>
* [lxd][runtime] Add LXD RuntimeArif Ali2023-10-152-1/+152
| | | | Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [snap] Add more items to collectArif Ali2023-10-151-1/+50
| | | | | Resolves: SET-313 Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [microceph][ubuntu] Collect mgr dataNikhil Kshirsagar2023-10-132-55/+88
| | | | Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com>
* [microceph][ubuntu] Collect ceph daemon commands in OSD nodeNikhil Kshirsagar2023-10-131-27/+43
| | | | | | | Now that https://github.com/canonical/microceph/issues/160 is fixed, gather the ceph daemon command outputs for OSD nodes. Signed-off-by: Nikhil Kshirsagar <nikhil.kshirsagar@canonical.com>
* [policy] Fix a couple of typosJose Castillo2023-10-121-2/+2
| | | | | | Fix a couple of typos. Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* Fix dynaconf obfuscation and add AUTH_LDAP_BIND_PASSWORDDr. Jason Breitweg2023-10-101-3/+6
| | | | | | | | | | | | | | Signed-off-by: Dr. Jason Breitweg <jason@breitweg.com> Fixed style issues Signed-off-by: Jason Breitweg jbreitwe@redhat.com Signed-off-by: Dr. Jason Breitweg <jason@breitweg.com> Fixed yet more linting errors Signed-off-by: Jason Breitweg jbreitwe@redhat.com Signed-off-by: Dr. Jason Breitweg <jason@breitweg.com>
* [powerpc] collect rtas_errd.log and lp_diag.log filesSathvika Vasireddy2023-10-101-0/+2
| | | | | | | | | | | | On power systems, rtas_errd daemon logs RTAS events to servicelog database. This daemon uses /var/log/rtas_errd.log file to store debug logs. Similarly, light path diagnostics log messages are written to /var/log/lp_diag.log file. Signed-off-by: Sathvika Vasireddy <sv@linux.ibm.com> Reviewed-by: Sourabh Jain <sourabhjain@linux.ibm.com>
* [docker] Fix runtime check_is_active sysroot considerationJake Hunsaker2023-10-101-3/+3
| | | | | | | | | | | A runtime's `check_is_active()` does not get passed a sysroot at any point, so the docker runtime abstraction should just use the policy sysroot directly. Additionally, `check_can_copy()` should be simplified to simply return the already-determined active state of the runtime. Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
* [cirrus] bump foreman versionsPavel Moravec2023-10-102-5/+5
| | | | | | | | | Bump tested foreman versions - use current latest (3.7) on Ubuntu and CentOS and previous 3.5 and 3.3 to match supported Satellite versions. This requires enabling proper module streams for foreman-3.3 as well. Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [filesys] skip filesystems picked up by other pluginsNathaniel Clark2023-10-041-1/+15
| | | | | | Skip /proc/fs/FS for FS picked up by other plugins Signed-off-by: Nathaniel Clark <nclark@whamcloud.com>
* [powerpc] Collect nvdimm logsSourabh Jain2023-10-041-0/+3
| | | | | | | | | | | | | | | Run an opal-prd command to collect nvdimm NV DIMM info which includes: - Number of NV DIMMs - Active slot - Falsh lifetime - Serial Number - Runtime The data generated by the opal-prd for NV DIMM info command is usually around 1K. Signed-off-by: Sourabh Jain <sourabhjain@linux.ibm.com>
* [firewall_tables] Gather nft rule handle numbersMichael Colombo2023-10-041-1/+1
| | | | | | | Added -a to 'nft list rulest' command to add handle numbers to the gathered output Signed-off-by: Michael Colombo <mcolombo@redhat.com>
* [global] Fix alerts for mixing implicit and explicit returnsJake Hunsaker2023-10-0118-25/+57
| | | | | | | | | This commit resolves the alerts generated by GitHub code scanning for mixing implicit and explicit returns. In most cases this is simply changing empty or implicit returns to `return None`, while in a few other places the return logic is slightly reworked. Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
* [git] Ignore DS_Store files from FinderJake Hunsaker2023-09-291-0/+1
| | | | | | | Updates .gitignore to exclude .DS_Store files that Finder may drop in the tree. Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
* [networking] obfuscate password in netplanArif Ali2023-09-281-0/+8
| | | | | resolves: #3365 Signed-off-by: Arif Ali <arif.ali@canonical.com>
* [foreman] collect schema_migrations tableEvgeni Golov2023-09-251-0/+1
| | | | Signed-off-by: Evgeni Golov <evgeni@golov.de>
* [openvswitch] Add DPDK info commandsKevin Traynor2023-09-231-1/+5
| | | | | | | Add commands that report info about DPDK lcore usage, log levels and memory usage. Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
* [plugins,distros] Resolve code alerts for file not always closedJake Hunsaker2023-09-2314-110/+122
| | | | | | | | | | | | | | | | | | | Fix a number of outstanding "File is not always closed" errors highlighted by GitHub's code scanning. This involves (almost exclusively) changing such as: for line in open('/some/file', 'r').read().splitlines(): To using a context manager to ensure the file is always closed: with open('/some/file', 'r') as sfile: for line in sfile.read().splitlines(): ...or something similar. Overall the logic of what a specific block of code was doing (mostly in plugins) is not changed however. Signed-off-by: Jake Hunsaker <jacob.r.hunsaker@gmail.com>
* [microshift] add kube-apiserver audit logs to collection pathJon Cope2023-09-191-0/+1
| | | | | | do not perform redundant dir-exists check Signed-off-by: Jon Cope <jcope@redhat.com>
* [microshift] Add /etc/microshift file copy specPablo Acevedo Montserrat2023-09-191-0/+3
| | | | Signed-off-by: Pablo Acevedo Montserrat <pacevedo@redhat.com>
* [virsh] Add --all for virsh pool and netnashok@redhat.com2023-09-181-1/+2
| | | | | | | Adding --all will also get inactive pools/net. This would be helpful when working with a pool/net which are failing to start. Signed-off-by: Nijin Ashok <nashok@redhat.com>
generated by cgit (git 2.34.1) at 2024-11-10 06:21:26 +0000