aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [pulp] Add new pluginJake Hunsaker2019-03-211-0/+72
| | | | | | | | | | Adds a new plugin for pulp as part of the effort to port the foreman-debug script to native sos plugins. Related: #1525 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [candlepin] Add new pluginJake Hunsaker2019-03-211-0/+47
| | | | | | | | | | Adds a new plugin for Candlepin as part of the effort to port the foreman-debug script to native sos plugins. Related: #1525 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [katello] Add new pluginJake Hunsaker2019-03-211-0/+61
| | | | | | | | | | Adds a new plugin for katello as part of the porting of the foreman-debug script to sos plugins. Related: #1525 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [puppet] Collect facter, directory listing, and version outputJake Hunsaker2019-03-211-1/+17
| | | | | | | | | | | Adds collection of 'facter' and the puppet version in use. Additionally collect directory listings for puppet module dirs. Related: #1525 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ruby] Add new pluginJake Hunsaker2019-03-211-0/+28
| | | | | | | | | Adds a new plugin for the ruby runtime. Related: #1525 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [samba] Get information about the primary trustAndreas Schneider2019-03-211-0/+1
| | | | | | | Resolves: #1602 Signed-off-by: Andreas Schneider <asn@cryptomilk.org> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [samba] Use verbose names and get testparm output firstAndreas Schneider2019-03-211-3/+3
| | | | | | | Related: #1602 Signed-off-by: Andreas Schneider <asn@cryptomilk.org> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [apache] honour --log-sizePavel Moravec2019-03-211-22/+20
| | | | | | | | | Honour --log-size option for collecting logs and dont use the hardcoded 5/15MB size limit. Resolves: #1203 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [kubernetes] Collect vm/vmi information if CNV is in useJake Hunsaker2019-03-211-0/+5
| | | | | | | | | | If CNV is installed ontop of the base kubernetes installation, collect output for the 'vms' and 'vmis' resources as well as virtctl. Resolves: #1599 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack_neutron] skip collecting /var/lib/neutron/lockPavel Moravec2019-03-211-0/+5
| | | | | | | | | Since the directory can have huge number of less important files, don't collect but just list them. Resolves: #1598 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [systemd] avoid collecting /dev/nullBryan Quigley2019-03-211-0/+1
| | | | | | | | | | | | | | | | | | Avoid "collecting" /dev/null as part of the systemd plugin. In my testing that was the only plugin that brought it in. Creating these character devices makes it harder to manage (delete) extracted sosreports without more permissions. This supersedes #1469 which removed the ability to "collect" char devices. As we only have one example today, let's just exclude that. Resolves: #1595 Signed-off-by: Bryan Quigley <bryan.quigley@canonical.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [nfs-ganesha] Update nfs-ganesha service log locationsSoumya Koduri2019-03-211-2/+1
| | | | | | | | | | nfs-ganesha service logs are now located at "/var/log/ganesha/" directory by default. Updating the same in sos plugin. Resolves: #1593 Signed-off-by: Soumya Koduri <skoduri@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [systemd] List-jobs already addedBryan Quigley2019-03-211-1/+0
| | | | | | | | | Removing the dupicate entry Resolves: #1591 Signed-off-by: Bryan Quigley <bryan.quigley@canonical.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [dbus] Collect busctl outputJake Hunsaker2019-03-211-0/+4
| | | | | | | | | Adds collection of busctl list and status. Resolves: #1589 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [crio] Update plugin to use inspeecti/p commandsJake Hunsaker2019-03-211-17/+31
| | | | | | | | | | | | | | | | | Updates the crio plugin to use the 'inspecti' and 'inspectp' subcommands for images and pods respectively. Additionally filter out the socket deprecation warning from being iterated over. Adds collection of 'ps -v' and updates the journal unit to 'crio' instead of 'cri-o'. Now collects all of /etc/containers, and adds collection of crio files under /etc/sysconfig. Resolves: #1588 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [buildah] Collect --all separately for containers and imagesJake Hunsaker2019-03-211-7/+11
| | | | | | | | | | | | | Adds a separate collections of --all for both the containers and images lists. Also, small fix to rename a variable to avoid using the same variable name twice. Resolves: #1582 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [podman] Correct command collection, add volume info collectionJake Hunsaker2019-03-211-12/+29
| | | | | | | | | | | | | | | | | | | | | | | Updates the podman plugin to collect volume information such as 'volume ls' and 'volume inspect' for every reported volume. Additionally, corrects the inspect iteration to not potentially call 'podman logs' on image IDs, only container IDs. The image inspect commands will also now use the image name unless either the name or tage includes 'none', in which case the image ID is used. Removes the attempt to collect a podman journal - podman does not log to a journal unit. Finally adds '--all' to the stats command, as podman requires this option when not specifying specific containers. Resolves: #1579 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [mysql] Collect galera cluster node stateJose Castillo2019-03-211-0/+1
| | | | | | | | | | | | | | | | | | | This patch collects the galera mysql node state stored in the following file: /var/lib/mysql/grastate.dat It contains details of the galera node like version, uuid, seqno, and current value in that node of safe_to_bootstrap. The idea is that for a galera mysql cluster, we'll get this status file from each node in the sosreport, and we'll have a fuller picture of the issue. Resolved: #1577 Signed-off-by: Jose Castillo <jcastillo@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [nvme] Adjust per-device command collectionJake Hunsaker2019-03-211-9/+15
| | | | | | | | | | | | | Removes 'nvme list' from repeated collection, instead only capturing it once and further collection 'nvme list-subsys'. Additionally adds per-device collection of smartctl output for further SMART information than what is provided by smart-log, and get-ns-id. Resolves: #1576 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [rasdaemon] Add new pluginJake Hunsaker2019-03-211-0/+35
| | | | | | | | | | Adds a new plugin for rasdaemon - a monitor for platform Reliability, Availability, and Serviceability (RAS) reports. Resolves: #1575 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [memory] Add lsmem collectionJake Hunsaker2019-03-211-1/+2
| | | | | | | | | | Adds collection of lsmem output for all memory blocks to the memory plugin. Resolves: #1573 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [apache] Collect config summary via apachectl -SJake Hunsaker2019-03-211-1/+4
| | | | | | | | | Adds collection of apachectl -S for quick config summary data Resolves: #1572 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [postgresql] Use postgres 10 scl if installedJake Hunsaker2019-03-211-3/+14
| | | | | | | | | | | | | | Updates the plugin to check if the specified SCL is running, as some systems may have multiple SCL versions installed, but only one will be running at a time. We now use the running version for a pgdump. This is primarily aimed at RHV environments as 4.3 and later use version 10. Resolves: #1562 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [python] check dist_version only on RHELPolicyPavel Moravec2019-03-211-1/+3
| | | | | | | | | Call /usr/libexec/platform-python only on RHEL (of version bigger than 7). Resolves: #1605 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [plugins] remove reference to xmlreportPavel Moravec2019-03-211-4/+0
| | | | | | Resolves: #1604 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [crio] import all plugins referred by CRIO classPavel Moravec2019-03-211-1/+1
| | | | | | Resolves: #1606 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [manageiq] import os.environ variable properlyPavel Moravec2019-03-211-4/+3
| | | | | | | | | os.environ is a variable not a module, so "import os.environ" doesnt work, causing plugin isnt loaded. Resolves: #1607 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [networking] import os.listdir methodPavel Moravec2019-03-211-1/+2
| | | | | | Resolves: #1608 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [ovirt] fix LGTM warning about regular expressionsPavel Moravec2019-03-201-4/+2
| | | | | | | | caret at the beginning of line, dollar at the end Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [iprconfig,sapnw] fix nested loops using same variablePavel Moravec2019-03-202-6/+6
| | | | | | | | Nested loops shall not use same variable name as their iterator. Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [general] remove duplicit module importsPavel Moravec2019-03-204-10/+5
| | | | | | Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [filesys,ldap,systemd] fix implicit string concatenationPavel Moravec2019-03-203-3/+3
| | | | | | Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [selinux] string comparison using !=Pavel Moravec2019-03-201-1/+1
| | | | | | Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [general] remove unused module importsPavel Moravec2019-03-2017-30/+7
| | | | | | Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [policies] ibmkvm: remove unreachanble codePavel Moravec2019-03-201-2/+0
| | | | | | Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [sosreport] remove XML reportsPavel Moravec2019-03-201-112/+1
| | | | | | | | Abandon dead feature not used for years. Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [gluster] remove noop assignmentPavel Moravec2019-03-201-1/+0
| | | | | | | | last_line redefined before any further refrence Relevant to: #1559 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [manageiq] Collect postgres and tower logsPavel Moravec2019-03-201-52/+26
| | | | | | | | | | | | Collect /var/log/tower.log for embedded Ansible Tower deployments. Simplify lists of configs and logs being collected by using few filemasks. If APPLIANCE_PG_DATA variable is set, collect postgres logs and config. Resolves: #1544 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [nvme] Capture /etc/nvme/discovery.confJose Castillo2019-03-201-0/+1
| | | | | | | | | | | | | The file /etc/nvme/discovery.conf is used by the command 'nvme discover' to send Get Log Page requests to a NVMe-over-Fabrics Discovery Controller. This plugin ensures that the sosreport captures the configuration, if it exists. Resolves: #1539 Signed-off-by: Jose Castillo <jose.mfcastillo@gmail.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [lvm2] Use metadata_read_only=1 for lvm commandsNir Soffer2019-03-201-8/+12
| | | | | | | | | | | | | | | | | We use locking_type=0 to ensure that we can collect data even if another process holds a conflicting lock. This introduce a small risk since even the read-only commands we use may attempt to recover on-disk data in some cases. The risk is bigger when running lvm2 plugin in oVirt system, when LVM is used on shared storage, accessible from many hosts. Add metadata_read_only=1 to ensure that no command can attempt to do unwanted on-disk changes. Fixes: #1533 Resolves: #1538 Signed-off-by: Nir Soffer <nsoffer@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [docker] add upstream package names to RedHatPluginJake Hunsaker2019-03-201-1/+2
| | | | | | | | | | Adds the upstream docker packaging to the packages list for the RH family of distributions. Related: #1580 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [docker] do not collect 'system df' by defaultJake Hunsaker2019-03-201-2/+2
| | | | | | | | | | Gates collection of 'docker system df' behind the docker.size option as this command can take a long time to run just like 'ps -as'. Related: #1580 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [docker] Correct commands, collect volume infoJake Hunsaker2019-03-201-15/+28
| | | | | | | | | | | | | Fixes the potential for 'docker logs' to be called against images and adds volume collection for docker like #1579 does for podman. Changes image inspections to use the image name, unless the name or tag is 'none' in which case the image ID is used. Resolves: #1580 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [config] update example sos.confBryn M. Reeves2019-03-201-4/+6
| | | | | | | | | Remove plugin options that no longer exist, and add more examples of global sosreport options that may be set in the config file. Related: #1530 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] handle non-valued and counter options in sos.confBryn M. Reeves2019-03-201-2/+20
| | | | | | | | | | | | | | | Command line arguments that are non-valued ("store_true"), or that have counter semantics need special handling when read from the configuration file (since optionless values *do* have an explicit value in the file, in order to meet .ini file structure, and since counter values are represented in the file as an integer). Process these options specially and return them in a form that is correctly parsed by the ArgumentParser. Related: #1530 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] support all command line options in /etc/sos.confPavel Moravec2019-03-202-112/+154
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - SoSOptions class changes: - from_file method to load load options from file - encapsulation of particular options into __init__ method - updated merge logic to replace non-default values only - sosreport.py changes: - _parse_args (renamed to _get_parser) newly returns just the ArgumentParser object as parsing itself is done elsewhere - SoSOptions are built in ordering: - defaults from parser (with emtpy cmdline) loaded - cmdline options replace defaults - config file known even now - config.file options replace defaults until already set by cmdline (i.e. until SoSOptions marked them as nondefault) - presets can be known even now - options from given preset replace defaults, until the options are already updated from cmdline or config.file - some extra logging/formatting added - some extra comments added Fixes: #855 Resolves: #1530 Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugins] Preserve data type of a plugin option to the default onePavel Moravec2019-03-201-1/+11
| | | | | | | | | | | | | | Some plugin options rely on the default data type that can be automatically overwritten by Python (e.g. from str to int). The commit preserves the original data type to ensure implicit data type specified in option definition is always used. Resolves: #1526 Related: #1597 Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [policy] Add a 'none' preset to Policy to override automatic presetsJake Hunsaker2019-03-191-1/+12
| | | | | | | | | | | | | | | | | Adds a 'none' preset to the base Policy class that can be used to override any distro-specific presets that may be otherwise automatically loaded when a bare 'sosreport' is run due to the presence of certian packages. In other words, using '--preset=none' will cause sosreport to run identically to how sosreport ran before presets were introduced. Additionally this 'none' preset is now the default preset returned by the Policy class if probe_preset() is not overridden by individual policies. Resolves: #1537 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [chrony] Adding support for Debian and UbuntuNick Niehoff2019-03-191-6/+22
| | | | | | | | | | Chrony is also supported on Debian and Ubuntu, but is configured slightly differently. Resolves: #1536 Signed-off-by: Nick Niehoff <nick.niehoff@canonical.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [manpages] Fix path for docs in /usr/share/docJose Castillo2019-03-191-1/+1
| | | | | | | | | | | | | | | | In the Makefile we have specified the path for sosreport docs as: install -m644 AUTHORS README.md $(DESTDIR)/usr/share/$(NAME)/. And NAME is 'sos'. But in the manpage we have the path as /usr/share/doc/sosreport. This fix removes the path and directs to the package documentation. Fixes: #1531 Resolves: #1532 Signed-off-by: Jose Castillo <jose.mfcastillo@gmail.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>