| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Fixes #395.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Allow the user to fine-tune the set of enabled plugins using the
-o (--only-plugins) and -n (--skip-plugins) options. These add
and remove plugins from the set defined by the active profile(s).
Both options may be specified multiple times, or given lists of
plugins to operate on. E.g.:
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add simple profile support. Plugins can define a 'profiles'
attribute that contains a list of profile names that this plugin
belongs to. If a profile is given on the command line only plugins
belonging to that profile will run.
Two new command-line options are introduced in thie commit:
--list-profiles
--profile=NAME
Profiles can be combined with -l/--list-plugins in order to see
the effect of the profile on the set of enabled plugins.
Fixes #247.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Plugin class docstrings should be a brief (42 characters or less)
description of the component the plugin supports. This is needed
since the class docstring is displayed in the table of enabled
plugins when running 'sosreport --list-plugins'.
Class docstrings should not describe the distribution a particular
class runs on; this is redundant since only plugins tagges with a
class matching the running policy will be displayed.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add an initial profile tag list to each plugin base class. A tag
list is a tuple of strings indicating the set of profiles that
should select this plugin, e.g.:
plugin_name = "qux"
profiles = ('system', 'storage', 'foo)
A follow up patch will add infrastructure to use these tag lists
to allow the user to select a logically-related set of plugins by
specifying a profile on the command line:
# sosreport -v --batch --profile=storage
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
sosreport does not accept commas, period and alphas in the case
id, the patch fixes the same.
Change the ticket number field to case id as it now accepts
alpha numerals as well but maintain the '--ticket-number' option
for compatibility.
Signed-off-by: Aruna Balakrishnaiah <aruna@linux.vnet.ibm.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
| |
Add the GUI and main abrt packages to the plugin's packages list.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
| |
Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
| |
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(-)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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(-)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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(+)
|
|
|
|
|
|
|
|
|
| |
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 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>
|
|
|
|
|
|
|
|
|
| |
Useful in Openstack environments.
Fixes #384
Signed-off-by: Lee Yarwood <lee.yarwood@gmail.com>
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
This splits out the ovs collection data from the Neutron plugin.
Fixes #328
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
Collect 'nmcli con show active' and 'nmcli dev show'.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
| |
Fixes #377.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
| |
Fixes #371.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
| |
Fixes #376
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
| |
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>
|