aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [ipmi] add new pluginBryn M. Reeves2016-02-031-0/+39
| | | | | | | | | | | | | | | | | Add a plugin to collect hardware data from IPMI-enabled hosts: ipmitool sel info ipmitool sel log ipmitool sensor list ipmitool chassis status ipmitool fru print ipmitool bmc info ipmitool sdr info Fixes: #756. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [logs] collect journalctl --disk-usagePavel Moravec2016-02-011-0/+1
| | | | | | Resolves: #750 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [rpm] move extra rpm fields to a separate fileBryn M. Reeves2016-02-011-7/+22
| | | | | | | | | | | | | | | | | | | | | | | The installed-rpms file is intended as a quick list of all installed packages for distributions using the RPM package manager: the contents have been fairly stable over several major releases and are formatted to be human-readable in a standard 80x25 terminal without needing to filter or sort. The addition of several large fields over the last year has led to some significant bloat however: python-ply-3.6-2.fc23.noarch Thu Oct 8 10:55:19 2015 1444298119 Fedora Project buildvm-09.phx2.fedoraproject.org 89021503050055d3387032474cf834ec9cba0108f41f10009231fd54252a459cabaccf2b68f39b7f2ab5474c0f29906494593c58808f4ebcc9cb1910921ddfd0fcd02c8b394af5f07915166499eff319d128fae9850363ffb5d7977adfa5c2b298ec1818975c16dd95721d0a6e3a5fd1de8bcb3d871e5a1bf2b2d39a15c1c53c5936863835e944e4a087bb982448420d852557fcfc81965ea2ecd7d9b1d6783d4e2bd2a57418ebe4ca92e1fec47c8b78864fd3fa81bab4b5766c04f92af7fd61452ad0063211edf390e075393ee3f6e019593bb230c29f134d1511c719ade1dd533db787c8e79ee77219bd8415a821bdb4e516e3c44e6fa9d25627e5157f1460a3599a183b1579077a51651fc34a734ee67a63bdeae42dbaf8429bc73543938cc86e8a92150e6c2de9bd7db1ae84d4938399388a445d0dfc0410b3c7f0c50399fb85ea8ec5c64e4138111bf394188b65f0e8013ed4a23f2973df663c4811ff02312cd57e3293fcc728d6d6c9951351fd0cfea1a6d4e90d5ff38be4858eb7f308efa5793f3955d641baf6cdc5495cfba0228e03af8bde85ffbb955766c6e31c6cbffe40539ca794806515198f44176680c17431720f4de00c4bf695dc54c53225698b45d8ac5b139e46503015f2456b420aeb5fab9a7cef12abd7308e410687771ba644b10ae680c21cacc40c71c5ab0159c2e021931695dee20bdc9d9c957e097c9a800634b88d60ad9c0a554bac3a5ca97b0f63538b9808 RSA/SHA256, Tue Aug 18 14:51:44 2015, Key ID 32474cf834ec9cba vs. python-ply-3.6-2.fc23.noarch Thu Oct 8 10:55:19 2015 Revert the installed-rpms file back to the original format and add a new file, package-data in the rpm plugin's commands directory that contains these additional rpm query fields. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [utilities] Support glob expansion in command argumentsShane Bradley2016-02-011-1/+10
| | | | | | | | | | | | | | | | | | | | | Historically, when calling an external process, sos relied on the use of a shell to perform glob expansion on arguments passed to the command. This potentially exposes sos and its children to the entire gamut of shell expansion and all the problems associated with it and this is part of the reason that use of a shell in callouts was disabled in commit 46b6c3d (Issue #253). While full shell syntax is undesirable for many reasons the use of globs to express path arguments to external commands is very useful and widely applied: restore this limited support for shell-like syntax using the python glob.glob module and expand any argument that is a valid glob into its resulting arguments. Fixes: #752 Signed-off-by: Shane Bradley <sbradley@redhat.com>
* [archive] remove hand-rolled compression shell outBryn M. Reeves2016-02-011-12/+9
| | | | | | | | | | The archive classes are not special snowflakes: they do not need their own (buggy) version of sos_get_command_output(): replace the hand-rolled Popen() call with the sos.utilities version. Really Fixes: #742. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Merge pull request #744 from sosreport/sbradley-rhn_managled_passwordshane bradley2016-01-221-2/+2
|\ | | | | [redhat] A managled rhn systemid will crash sosreport.
| * [redhat] A managled rhn systemid will crash sosreport.Shane Bradley2016-01-221-2/+2
|/ | | | | | Fixes: #742 Signed-off-by: Shane Bradley <sbradley@redhat.com>
* [openvswitch] gather database in 'list' formatBryn M. Reeves2016-01-151-1/+1
| | | | | | | | | | | This changes database dump collection format to 'list' which is a more readable and easy to parse. Based on a change suggested by Prasad Mukhedkar. Resolves: #676. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [utilities] remove unused hashlib importAbhijeet Kasurde2016-01-141-1/+0
| | | | Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* [sosreport] report correct final path with --buildBryn M. Reeves2016-01-131-1/+3
| | | | | | | Ensure the correct path (in the system temporary directory) is reported when the --build option is used. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openshift] Obfuscate LDAP bind passwordsPep Turró Mauri2015-12-181-0/+5
| | | | | | | | | | | | OpenShift broker and console can be configured with different authentication backends. If LDAP is being used with a server that requires authentication for search the configuration files will capture the credentials of the BindDN. Fixes: rhbz#1227462 Fixes: #673. Signed-off-by: Pep Turró Mauri <pep@redhat.com>
* [powerpc] fix missing commas in copy spec listMartin Frodl2015-12-181-2/+2
| | | | | | Fixes: #671. Signed-off-by: Martin Frodl <mfrodl@redhat.com>
* [ovirt] Remove database password from AAA_JDBC profile filesNijin Ashok2015-12-181-1/+2
| | | | | | | | | | | | | | | | | Ovirt AAA_JDBC stores database password in plain text under /etc/ovirt-engine/aaa/*.properties file. It contains password in the format: config.datasource.dbpassword=<password> The engine PostgreSQL user password and the custom build domain's user password exist in plain text format in these files. The commit removes these password from the sosreport. Fixes: #666. Signed-off-by: Nijin Ashok <nashok@redhat.com>
* [apache] fix files list styleBryn M. Reeves2015-12-181-3/+5
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [apache] collect list of modulesPavel Moravec2015-12-181-0/+4
| | | | | | | | Collect output of "apachectl -M" for that Resolves: #691 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [md] Add collection of per-device and global device parametersKenneth Koski2015-12-181-1/+3
| | | | | | | | | | | | | | For more information about these parameters, see the md man page: http://linux.die.net/man/4/md The kernel.org documentation also has some information about these parameters: https://www.kernel.org/doc/Documentation/md.txt Fixes #700. Signed-off-by: Kenneth Koski <mechaxl@gmail.com>
* [apache] Added collection for JBoss Web ServerHisanobu Okuda2015-12-161-3/+28
| | | | Signed-off-by: Hisanobu Okuda <hokuda@redhat.com>
* [haproxy] Collect /etc/haproxy/conf.d/*Michele Baldessari2015-12-141-0/+1
| | | | | | | | | | Some openstack installations split out the configuration files via an "include conf.d/*.cfg" directive. Add the whole directory in order to collect that as well. Fixes: #674. Signed-off-by: Michele Baldessari <michele@acksyn.org>
* [sosreport] clean up private temporary directoryBryn M. Reeves2015-12-101-1/+6
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [multipath] collect output of 'multipathd show config'Bryn M. Reeves2015-12-081-1/+2
| | | | | | Resolves: #697. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] prepare report in a private subdirectoryBryn M. Reeves2015-12-041-23/+77
| | | | | | | | | | | | | | | | | | | | | | | To avoid file creation races in shared temporary directories like /tmp and /var/tmp use a private (0700) subdirectory to build the FileCacheArchive and subsequent archive and compressed archive files: only create a file in the containing directory when it can be done as a single atomic rename. This prevents sos from writing to an arbitrary location under the control of another user: a malicious user could steal data or over write files in /etc resulting in a local privilege escalation. There remains a further race since once the archive name is known the checksum file name becomes predictable: as the checksum file is also prepared in the subdirectory and moved into place the result is always either success or an error that is reported to the user. The correct checksum value is still reported to the user via the terminal. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] move archive checksumming to sosreportBryn M. Reeves2015-12-042-22/+26
| | | | | | | | | | | | | | | | Although the digest algorithm is policy controlled the actual mechanism to checksum the archive does not belong in the policies module: historically this was done to keep the code that calculates the checksum close to the UI code that reports it. Move the calculation to the main SoSReport class's final_work() method and add a 'checksum' argument to the display_results() method so that the value can be reported. In future it may make sense to push the checksum code directly into the archive class. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [policies] refactor Policy.display_results() argsBryn M. Reeves2015-12-042-13/+23
| | | | | | | | | Pass explicit archive and build directory arguments to the Policy.display_results() method rather than a single path name argument and a boolean to indicate whether it is an archive file or a build directory path. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [policies] move hash determination to policiesBryn M. Reeves2015-12-043-20/+16
| | | | | | | | | | | | | It's crazy having the Policy classes call a function in the utilities module only to have that function then load the Policy module, call policy.get_preferred_hash_algorithm() and then test the result. Get rid of the get_hash_name() function in the utilities module and simplify the calls in the policies module to obtain the preferred hash name. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [build-rpm] fix rpm build for sphinx docs directoryBryn M. Reeves2015-12-041-1/+2
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [ceph] Collect (parts of) /var/lib/ceph and /var/run/cephPavel Moravec2015-11-271-1/+6
| | | | | | | | | | Collect the two directories without some forbidden paths. Reorder self.add_forbidden_path(..) to have some logical ordering there. Resolves: #693 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [insights] stray whitespace fixes from previous commitBryn M. Reeves2015-10-151-3/+3
|
* [insights] minor style fixesBryn M. Reeves2015-10-151-7/+8
|
* [insights] plugin for Red Hat Access Insights dataJeremy Crafts2015-10-151-0/+42
| | | | | | Closes: #663. Signed-off-by: Jeremy Crafts <jcrafts@redhat.com>
* [archive] Don't try to overwrite dangling symlinksJan Grant2015-10-151-1/+1
| | | | | | | | | | | | | The test for a symlink already existing would fail if the symlink target did not exist; this will be the case when a symlink points to a directory (which is not copied). Fix up to use os.lexists instead. Closes: #660 Closes: #661 Signed-off-by: jan grant <jang@ioctl.org>
* [openstack_ironic] Limit log collection to .log files by defaultLee Yarwood2015-10-151-1/+8
| | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_sahara] Limit log collection to .log files by defaultLee Yarwood2015-10-151-3/+8
| | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_trove] Limit log collection to .log files by defaultLee Yarwood2015-10-151-4/+10
| | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_neutron] Limit log collection to .log files by defaultLee Yarwood2015-10-151-6/+10
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #446 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_keystone] Limit log collection to .log files by defaultLee Yarwood2015-10-151-4/+8
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #445 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_horizon] Limit log collection to .log files by defaultLee Yarwood2015-10-151-3/+10
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #444 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_heat] Limit log collection to .log files by defaultLee Yarwood2015-10-151-5/+11
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #443 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_glance] Limit log collection to .log files by defaultLee Yarwood2015-10-151-5/+11
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #442 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_cinder] Limit log collection to .log files by defaultLee Yarwood2015-10-151-4/+8
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #441 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_ceilometer] Limit log collection to .log files by defaultLee Yarwood2015-10-151-5/+9
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #440 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_swift] Limit log collection to .log files by defaultLee Yarwood2015-10-151-3/+9
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #448 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_*] fix setup() regressions introduced in 1699eabdLee Yarwood2015-10-158-20/+20
| | | | | | | | | | 1699eabd partially renamed various OpenStack plugin classes but was not complete when it was pulled in leading to various openstack plugins failing during setup. Resolves: #655 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [kernel] Add collection of /sys/kernel/slabsLouis Bouchard2015-10-151-0/+1
| | | | | | | | | | Needed when investigating slab related issue to identify how slabs are merged together. ls -lt shows symlink of merged slabs. Closes: #653, #654. Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
* [qpid_dispatch] code cleanup and adding authorshipPavel Moravec2015-10-151-9/+8
| | | | | | | | | Merged identical code into one loop iterating over ["ssl-certificate", "ssl-key", "ssl-trustfile"]. Added authorship. Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [archive] Force decoding if content is bytesLouis Bouchard2015-10-151-0/+2
| | | | | | | | | | | | | | | When content is coming from the output of the tail() method, it is a bytes in python3 and a string in python2. Force the bytes value to be decoded otherwise the later write() function will fail and nothing will be written to the file. Conditional decoding is necessary as the strings in python3 do not have a decode method. Closes #586 Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
* [archive] Add test to highlight py3 specific errorLouis Bouchard2015-10-151-0/+14
| | | | | | | When add_string receives output from tail() it fails in py3 since tail() returns a byte object. Mimic this behavior. Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
* [openstack_sahara] capture logs and packages in split modePoornima2015-10-151-0/+4
| | | | | | | | | Adding two services openstack-sahara-api and openstack-sahara-engine with required "-api + -engine" RPM package. Fixes: #641. Signed-off-by: Poornima M. Kshirsagar pkshiras@redhat.com
* [qpid] Add port and SSL related optionsPavel Moravec2015-10-151-17/+37
| | | | | | | | | | | | | Four options added: - port for specifying non-default listening port (sometimes necessary also for SSL) - ssl-certificate and ssl-key for SSL - ssl as rather auxiliary option to force qpid-* tools to use SSL even without ssl-* options specified Resolves: #634, #635 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [qpid] Fix missing commas in add_cmd_outputPavel Moravec2015-10-151-2/+2
| | | | | | | | Commands "qpid-config exchanges" and "qpid-config queues" lack comma in the list. That causes the commands are not collected by sosreport and executed. Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [docker] collect journald logs for docker unitBryn M. Reeves2015-10-071-0/+1
| | | | | | Fixes: #609. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>