| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
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>
|
|\
| |
| |
| |
| |
| | |
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.
|
| |/
|/|
| |
| |
| |
| |
| |
| | |
- sos/plugins/azure: Microsoft Azure Agent
- sos/plugins/juju: Ubuntu JUJU
- sos/plugins/maas: Ubuntu MAAS
Signed-off-by: Adam Stokes <hackr@cypherbook.com>
|
|\ \
| |/
|/| |
Include new-line in shell_out
|
|/ |
|
|
|
|
|
|
| |
The TarFileArchive with SELinux does not pass the full test
archive_tests suite. Catch an uncaught exception when stat'ing a
non-existent path and do not allow empty paths in add_parent().
|
|
|
|
|
|
|
|
|
|
| |
Commands run by sos inherit the environment of the user running the
program. This includes locale settings meaning that collected
output is subject to local language, sorting and formatting
customisations that can be undesirable when post-processing or
parsing of the gathered data is required.
Force all external processes to use the 'C' locale.
|
|
|
|
|
|
| |
The TarFileArchive class currently doesn't allow recursive
directory addition using the add_file() interface - disable those
tests for now.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the start of the archive reworking to address Issue #86.
In this commit the Archive class hierarchy is separated from the
generic code in the utilities module and moved into its own
'archive' module. This is taken from the changes committed to
Jesse's directory_archive branch.
The next step is to decide how we will handle the problems
discovered with the in-line tarfile generation (and similar
problems affecting Zip files) and whether to revert to building
an in-filesystem tree (as Jesse's current branch does) or whether
to fix the problems with in-lining post-processing hooks.
|
|
|
|
|
|
|
|
|
|
|
| |
The method names for collecting external output are ugly and
awkward to type. Rename them to be more consistent with the file
collection API and easier to remember and type.
This change touches virtually every plugin class but is a very
simple search/replace (plugin changes were done with sed).
Fixes Issue #92.
|
|\
| |
| | |
Make Ubuntu/Debian depend on python-selinux (Closed GH:#93)
|