| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
The ksyms program is a 2.4ism that has not existed on distros for
years. Remove all reference to it from the kernel plug-in.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
The hardware plug-in contains code to check for the existence
of binaries before running them. This won't work if plug-ins are
providing non-absolute paths for executables and is redundant in
any case since the shell-out wrappers already do this and log
unavailable binaries. Also removes the workaround for lsusb
changing path in recent versions.
Signed-of-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
PATH changes caused an incorrect '/usr/bin/java' -> '/usrjava'
change in the general plug-in. Revert it.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
| |
The automated changes to remove absolute paths went a bit awry
with odd paths in /opt and other locations. Fix these up in the
emc plug-in.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|\
| |
| | |
Add basic cinder data to the plugin
|
| |
| |
| | |
Pull in /etc/cinder, cinder version and logs for cinder.
|
|\ \
| | |
| | | |
Fix unittests and more pep8 conforming
|
| | |
| | |
| | |
| | | |
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- In our unittest we defined 'is_installed' to be a bool when in fact we wanted
to verify if a package is installed via the _method_ is is_installed. Since
overriding that method with our variable definition we basically removed
any functionality of 'is_installed' method in Plugin class.
- Initializing a fake plugin to test was failing due to our global
logger not being initialized. Put a simple check in the library to
make sure the logger is defined no matter what the calling state does.
- Make import selinux conditional
- Add travis-ci support
- Rename README to utilize travis-ci build status and other markdown supported
features
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Now that policies have control of PATH there is no need for
plug-ins to try to guess the correct absolute path (or work
around cases where it is known to have changed).
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Make the PATH environment variable subject to Policy class
control. Policies may set PATH to an appropriately formatted
string for the platform. This will be exported to the environment
and used as the search path for binaries that sos modules execute.
Following this change all absolute paths can be removed from
plug-in code simplifying maintenance on multiple distributions
and distribution versions (e.g. Red Hat UsrMove and various
updates that relocate binaries such as lsusb).
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|/ /
| |
| |
| |
| | |
The selinuxconfig command has not been shipped for years and does
not exist on modern SELinux using distributions.
|
|\ \
| | |
| | |
| | |
| | |
| | | |
Conflicts:
sos/plugins/general.py
sos/plugins/juju.py
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fix policy and utility function, method and variable names to use
lower case underscored style instead of camelCase.
Related to Issue #112.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The name copy_stuff() isn't as descriptive now that the data may
be being streamed into an archive without being copied into a
temporary directory. Calling it collect covers both equally well.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There are lots of historical camelCase function and method names
in the plugin directory even though pep8 very clearly recommends
against these other than for external backwards compatibility.
Rename all the camelCased functions and methods and fix up the
main sosreport code, examples and tests to use the new names.
Fixes Issue #112.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|\ \
| | |
| | | |
Fix for classname inversion in general.py
|
| |/ |
|
|\ \
| | |
| | | |
Added plugin for landscape client on Ubuntu.
|
| | | |
|
| | | |
|
| |/
| |
| |
| | |
Ubuntu Advantage landscape client.
|
|\ \
| | |
| | | |
Fix super call to proper general plugin name
|
| |/
| |
| |
| |
| |
| |
| |
| | |
We were calling super on incorrect plugin class. This was a result of the
conforming name patch applied.
Also went ahead and capitalized general plugin for pep8 spec.
Signed-off-by: Adam Stokes <hackr@cypherbook.com>
|
|\ \
| |/
|/| |
Only specify the var/log/juju dir
|
|/
|
| |
Only the juju directory exists in /var/log/ so grab that rather than globbing for the files.
|
|\
| |
| | |
Port several plugins to Debian/Ubuntu
|
| |
| |
| |
| | |
Signed-off-by: Adam Stokes <hackr@cypherbook.com>
|
| |\
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Clean up variable names and description strings in the command
line parsing ahead of splitting up the SoSReport class into
interface and back-end components.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
'silent' is a poort name for an option that admits to not being
silent:
--silent Only display FATAL errors on stdout
Rename the option to '--quiet' instead and document it in the man
page. It's probably also worth either documenting the fact that
UI text is still printed when --quiet is given in non-batch mode
or changing interactive mode to only print essential prompts when
run with --quiet.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
| |
| |
| |
| | |
Indent paragraphs in the UI text to improve readability.
|
| |
| |
| |
| |
| |
| |
| | |
Went through a few plugins to make sure inherit distro plugins were keeping
same naming scheme as the rest.
Signed-off-by: Adam Stokes <hackr@cypherbook.com>
|
| |\ |
|
| | |
| | |
| | | |
Remove debugging pprint
|
| |/
| |
| |
| | |
Signed-off-by: Adam Stokes <hackr@cypherbook.com>
|
| |
| |
| |
| | |
Signed-off-by: Adam Stokes <hackr@cypherbook.com>
|
|/
|
|
|
|
|
|
| |
[ Louis Bouchard ]
- Ported several plugins to work across Red Hat, Debian
and Ubuntu.
Signed-off-by: Adam Stokes <hackr@cypherbook.com>
|
|\
| |
| | |
Fix UI text templating
|
| |
| |
| |
| |
| |
| |
| |
| | |
Make sure that the Fedora policy gets a chance to run by checking
for the absence of /etc/fedora-release in the RHEL policy and
fix the inheritance of FedoraPolicy.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The current boilerplate text in the policy class has inconsistent
formatting and does not make sense with some distribution names
(e.g. Red Hat Enterprise Linux).
Fix this by storing the string in paragraphs and formatting it
with textwrap and adding new tags to make the construction of the
message more flexible:
vendor
vendor_url
vendor_text
tmpdir
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
Add modules for Ubuntu cloud infrastructure
- Azure hypervisor support
- MAAS
- Juju
|
| |\
| |/
|/| |
|
|\ \
| | |
| | | |
Line 243 attempts to access non-imported regex
|
| | |
| | |
| | | |
This was a result from existing unittests.
|