aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [ipa] fix get_cmd_output_now usageBryn M. Reeves2014-09-151-1/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [hpasm] fix add_copy_specs() useBryn M. Reeves2014-09-151-1/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [anaconda] make password scrubbing more robustBryn M. Reeves2014-09-141-2/+2
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [abrt] trigger plugin on more packagesBryn M. Reeves2014-09-141-1/+1
| | | | | | Add the GUI and main abrt packages to the plugin's packages list. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] remove jython logging handler workaroundBryn M. Reeves2014-09-131-7/+2
| | | | | | | | | | | | The workaround in sosreport.py for log handlers persisting past logging.shutdown() on Jython causes further problems with leaking log handler messages in the non-build case (i.e. where we are calling the Archive's finalize() method). Fix this by removing the workaround since sos no longer targets the Jython runtime. Fixes: #393. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] re-order logging shutdownBryn M. Reeves2014-09-131-6/+10
| | | | | | | | | | | | | | | | | | | Currently we remove all log handlers and then add the log files to the archive. This can cause messages to leak to the console when running in --quiet mode: # sosreport -o lvm2 --batch --build --debug --quiet No handlers could be found for logger "sos" This happens because the Archive classes output to the 'sos' logger; removing the handlers before archiving the logs causes a default basicConfig to be used and generates the above message. Add the files to the archive and then remove the handlers. Fixes #393. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] log sosreport command lineBryn M. Reeves2014-09-131-0/+3
| | | | | | | | | | Log the command line given to sosreport in the report log: INFO: [sos.sosreport:setup] executing 'sosreport -o lvm2 --batch --build --debug' Fixes #392. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] ensure Plugin has an 'archive' memberBryn M. Reeves2014-09-121-0/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] remove unused itertools importBryn M. Reeves2014-09-121-2/+0
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] remove superfluous parens in Plugin.report()Bryn M. Reeves2014-09-121-2/+2
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] fix indentation in Plugin._collect_copy_specs()Bryn M. Reeves2014-09-121-2/+2
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] remove superfluous parens in Plugin.get_cmd_output_now()Bryn M. Reeves2014-09-121-1/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] remove superfluous parens in Plugin.check_ext_prog()Bryn M. Reeves2014-09-121-1/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] remove superfluous parens in Plugin._do_copy_path()Bryn M. Reeves2014-09-121-1/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] fix whitespace in Plugin._copy_symlink()Bryn M. Reeves2014-09-121-5/+4
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] remove superfluous parens in is_installed()Bryn M. Reeves2014-09-121-1/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [plugin] mark more interfaces privateBryn M. Reeves2014-09-122-37/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | Numerous Plugin methods are not expected to be called from outside the base class. Mark them private with a leading underscore: _copy_dir() _copy_node() _copy_symlink() _do_copy_path() _expand_copy_spec() _mangle_command() _make_command_filename() _is_forbidden_path() _collect_copy_specs() _collect_cmd_output() _collect_strings() A couple of these (_mangle_command() and _do_copy_path()) are invoked from the Plugin test suite; update the relevant cases to call the new names. Fixes #348. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Fix variable naming according to commentLouis Bouchard2014-09-121-2/+2
| | | | | Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [apt] Add apt-cache policy info for installed pkgLouis Bouchard2014-09-121-1/+7
| | | | | | | | apt-cache policy information for each package can be useful to identify the source of the installed package. Fixes #389 Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
* [ovirt_hosted_engine] new plugin for oVirtBryn M. Reeves2014-09-121-0/+85
| | | | | | | | | | | | | | | The oVirt Hosted Engine packages allow to run ovirt-engine inside a VM. This plugin provides info about the setup and the high availability daemons running such VM. Change-Id: I4380bbed5bab4dd7fd3472bf5d5d1265f1eed3e7 Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com> --- sos/plugins/ovirt_hosted_engine.py | 85 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 85 insertions(+) create mode 100644 sos/plugins/ovirt_hosted_engine.py
* [ovirt] nit: fixing bad indentationBryn M. Reeves2014-09-121-5/+5
| | | | | | | | | Change-Id: I5f570416d8f80d79589bf93854ece1b307dc60ae Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com> --- sos/plugins/ovirt.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)
* [ovirt] add jboss runtime configurationBryn M. Reeves2014-09-121-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In oVirt 3.5 jboss runtime config changed location. Add to the report: /var/lib/ovirt-engine-reports/jboss_runtime/config/ |-- ovirt-engine-reportsd_history | |-- current | | `-- ovirt-engine-reportsd.v1.xml | |-- ovirt-engine-reportsd.boot.xml | |-- ovirt-engine-reportsd.initial.xml | |-- ovirt-engine-reportsd.last.xml | `-- snapshot |-- ovirt-engine-reportsd-logging.properties `-- ovirt-engine-reportsd.xml /var/lib/ovirt-engine/jboss_runtime/config/ |-- ovirt-engine_history | |-- current | | `-- ovirt-engine.v1.xml | |-- ovirt-engine.boot.xml | |-- ovirt-engine.initial.xml | |-- ovirt-engine.last.xml | `-- snapshot |-- ovirt-engine-logging.properties `-- ovirt-engine.xml Change-Id: I15864ef5197ec1c42f4a68445d5586847e595bc2 Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com> --- sos/plugins/ovirt.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
* [ovirt] add ovirt-engine-reports configBryn M. Reeves2014-09-121-0/+1
| | | | | | | | | | | | | | | | | | | Add: /etc/ovirt-engine-reports |-- ovirt-engine-reports.conf.d | |-- 10-setup-jboss.conf | `-- 10-setup-protocols.conf `-- sso.properties to the report Change-Id: I2287258b9652aa9ebf7f60528c378f4eeb2179d3 Related-To: https://bugzilla.redhat.com/1086942 Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com> --- sos/plugins/ovirt.py | 1 + 1 file changed, 1 insertion(+)
* [ovirt] ensure /etc/rhevm is collectedBryn M. Reeves2014-09-121-1/+1
| | | | | | | | | The path /etc/rhevem may be a symbolic link. Include a trailing slash in the copy spec to ensure it is collectes. Based on a suggestion from Sandro Bonazzola. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [rpm] Add Signature tag to the output of 'installed-rpms'poornimakshirsagar2014-09-111-2/+3
| | | | | | | | RPM meta data consists of Signature,Packager,BuildHost,Vendor tags it is useful for identifying 3rd party packages present on a system. This patch adds the gpg and pgp signature columns to installed-rpms. Signed-off-by: Poornima M. Kshirsagar <pkshiras@redhat.com>
* [networking] Collect a list of active network namespaces.Lee Yarwood2014-09-051-0/+1
| | | | | | | | | Useful in Openstack environments. Fixes #384 Signed-off-by: Lee Yarwood <lee.yarwood@gmail.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [postgresql] ensure password option is initialised to known valueBryn M. Reeves2014-09-031-2/+2
| | | | | | | | | The value of self.get_option("password") is tested in the postgresql plugin's setup() method to determine if the user gave a password on the command line. Set the default value to False to ensure we get the expected comparison results. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [dbus] new dbus pluginAdam Stokes2014-08-222-1/+30
| | | | | | | | | Adds dbus plugin support for Fedora, Debian, and Ubuntu. Also pulls the `/var/lib/dbus/machine-id` out of general into this plugin. Fixes #350 Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [ubuntu] rename from ubuntu-support-statusAdam Stokes2014-08-221-6/+6
| | | | | | | | In an attempt to keep distro specific collection (ie ubuntu-support-status) create a general `Ubuntu` plugin to categorize this information. This will be the default for other distributions needing to do the same. Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [openvswitch] Add new pluginAdam Stokes2014-08-222-20/+47
| | | | | | | | This splits out the ovs collection data from the Neutron plugin. Fixes #328 Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [general] remove tree and ls capturesAdam Stokes2014-08-221-5/+0
| | | | | | | | | | Ubuntu specifically was seeing general plugin times of ~19 seconds which most of that time spent running `tree /var/lib`. Was decided to remove this capture output and to be re-evaluated of its importance at a later date. Fixes #374 Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [openstack-*,ubuntu_support_status] fix plugin_nameAdam Stokes2014-08-229-9/+9
| | | | | | | | | | | | | | | We previously fixed an issue where plugin names needed to have underscores in the actual filename otherwise they wouldn't be resolved. The `plugin_name` attribute was never updated to reflect that. This change fixes to match that of the filename so that when calling something like sosreport -o ubuntu_support_status Will work rather than fail with unable to find plugin errors. Fixes #379 Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [policies] Add dist_version to policy classAdam Stokes2014-08-224-8/+29
| | | | | | | | | | | Exposes a `dist_version` to be inherited by policy classes to provide distribution release numbers. For example, on Ubuntu calling this method would return with `14.04`. Fixes #216 Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [plugins] New ubuntu-support-status pluginJorge Niedbalski R2014-08-211-0/+36
| | | | | | | | | | This ubuntu-specific plugin exposes the ubuntu-support-status command for gathering package support status information. Fixes #378 Signed-off-by: Jorge Niedbalski R <niedbalski@gmail.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [networking] additional nmcli commandsBryn M. Reeves2014-08-211-0/+2
| | | | | | Collect 'nmcli con show active' and 'nmcli dev show'. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [networking] fix 'nmcli con show' usageBryn M. Reeves2014-08-211-1/+1
| | | | | | Fixes #377. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [networking] add 'brctl showmacs' outputBryn M. Reeves2014-08-211-1/+4
| | | | | | Fixes #371. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sosreport] handle KeyboardInterrupt while importing sosBryn M. Reeves2014-08-211-1/+4
| | | | | | Fixes #376 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [postgresql] obtain PGPASSWORD from the environmentBryn M. Reeves2014-08-211-6/+11
| | | | | | | | Allow the value of PGPASSWORD to be inherited from the environment. This prevents the password being exposed in command line parameters that may be visible to other users on the system. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sos] update version templateAdam Stokes2014-08-202-19/+19
| | | | | | | | Updated `__init__.py.in` to be pep8 compliant. Also updated the version string to match that of whats defined in `sos.spec`. The alpha release 1 was manually appended since our Makefile only deals in release version. Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [rpm] capture BuildHost tag for installed rpmsBryn M. Reeves2014-08-201-3/+3
| | | | | | | | | | | | The RPM BuildHost tag may be of use in identifying 3rd party packages present on a system. Based on a patch from Chandan Kumar. Fixes #368. Signed-off-by:Chandan Kumar chkumar@redhat.com Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [juju] Adds 'juju get' output, exposes export-mongodbJorge Niedbalski R2014-08-192-1/+54
| | | | | | | | | | This commit extends the plugin to include the resulting 'juju get ${service}' for each deployed service, also it exposes the export-mongodb option, which includes on the report a json file containing each data collection used by juju-core. Signed-off-by: Jorge Niedbalski R <niedbalski@gmail.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [python] add package check and make portableBryn M. Reeves2014-08-191-2/+6
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [python] new plugin for python runtimepoornimakshirsagar2014-08-191-0/+27
| | | | | | | | | Python runtime details are required for developers for various diagnostic and development purposes. Collect the Python version of the 'python' executable. Signed-off-by: Poornima M. Kshirsagar <pkshiras@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [fcoe] Add plugin to get information about FCoE.Jose Castillo2014-08-181-0/+38
| | | | | | | | | | | | | | | | | This plugin gets the configuration files present in the directory /etc/fcoe and the output of two commands: - 'fcoeadmin -i' shows information about all FCoE instances, and - 'fcoeadmin -s' shows information about all discovered Fibre Channel Forwarders (FCFs). Attempts to fix issue #369 Version 2: Added a line to check if the package fcoe-utils is installed as suggested by Adam Stokes. Signed-off-by: Jose Castillo <jcastillo@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [lvm2] Add pe_start to the output of 'pvs'Jose Castillo2014-08-151-0/+1
| | | | | | | | | This patch adds the column pe_start (alignment offset) to the output of the 'pvs' command in the lvm2 plugin. Solves #331 Signed-off-by: Jose Castillo <jcastillo@redhat.com>
* [ovirt] add rhevm packagesBryn M. Reeves2014-08-121-1/+5
| | | | | | | | | Downstream packages for ovirt have branding patches renaming them with rhevm prefix instead of ovirt-engine. Signed-off-by: Sandro Bonazzola <sbonazzo redhat com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [activemq] inherit from base class properlyAdam Stokes2014-08-121-1/+1
| | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [docker] capture logs of running containers, add ubuntu supportAdam Stokes2014-08-111-7/+40
| | | | | | | | | Query `docker ps` and capture log output from current running containers. Additionally, add support for Ubuntu systems and address a nuance where Ubuntu systems use a different binary name for docker since a previous package already owns the docker binary name. Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [plugin] make Plugin logging methods privateBryn M. Reeves2014-08-099-58/+58
| | | | | | Related: #348. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>