aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Remove obsolete ksyms check from kernel plug-inBryn M. Reeves2013-04-161-2/+0
| | | | | | | 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>
* Remove PATH workarounds from hardware plug-inBryn M. Reeves2013-04-161-13/+6
| | | | | | | | | | | 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>
* Fixup PATH breakage in general plug-inBryn M. Reeves2013-04-161-1/+1
| | | | | | | 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>
* Fixup PATH breakage in emc plug-inBryn M. Reeves2013-04-161-59/+59
| | | | | | | | 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>
* Merge pull request #108 from ivoks/patch-1Bryn M. Reeves2013-04-101-1/+16
|\ | | | | Add basic cinder data to the plugin
| * Add basic cinder data to the pluginivoks2013-03-281-1/+16
| | | | | | Pull in /etc/cinder, cinder version and logs for cinder.
* | Merge pull request #118 from battlemidget/unittest-fixes-1Bryn M. Reeves2013-04-1028-53/+92
|\ \ | | | | | | Fix unittests and more pep8 conforming
| * | More pep8 naming updatesAdam Stokes2013-04-085-6/+6
| | | | | | | | | | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * | - Remove unused DEPS envAdam Stokes2013-04-041-3/+0
| | | | | | | | | | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * | Include coverage module during CI testsAdam Stokes2013-04-042-1/+2
| | | | | | | | | | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * | remove python 2.6 build support and allow failure of python 3.x testsAdam Stokes2013-04-041-1/+6
| | | | | | | | | | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * | More pep8 classname stylingAdam Stokes2013-04-0415-25/+25
| | | | | | | | | | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * | Fix unittests and more pep8 conformingAdam Stokes2013-04-048-22/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - 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>
* | | Remove absolute paths from plug-insBryn M. Reeves2013-04-0958-306/+306
| | | | | | | | | | | | | | | | | | | | | | | | 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 PATH policy controlledBryn M. Reeves2013-04-093-5/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | Remove obsolete command from selinux pluginBryn M. Reeves2013-04-091-1/+0
|/ / | | | | | | | | The selinuxconfig command has not been shipped for years and does not exist on modern SELinux using distributions.
* | Merge branch 'bmr-pep8ify-names'Bryn M. Reeves2013-04-04121-1154/+1154
|\ \ | | | | | | | | | | | | | | | Conflicts: sos/plugins/general.py sos/plugins/juju.py
| * | Make more names pep8 compliant across the treeBryn M. Reeves2013-03-2851-268/+268
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
| * | Rename copy_stuff() to collect()Bryn M. Reeves2013-03-283-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | 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>
| * | Rename functions and methods in plugins to comply with pep8Bryn M. Reeves2013-03-28114-896/+896
| |/ | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | Merge pull request #113 from karibou/masterBryn M. Reeves2013-04-030-0/+0
|\ \ | | | | | | Fix for classname inversion in general.py
| * | Fixed Class name inversionLouis Bouchard2013-04-021-2/+2
| |/
* | Merge pull request #111 from rubeon/masterBryn M. Reeves2013-04-031-0/+72
|\ \ | | | | | | Added plugin for landscape client on Ubuntu.
| * | Changed classname to match style guidelines.Eric Williams2013-03-291-1/+1
| | |
| * | Renamed landscape_client to landscape so it looks better in the plugin list.Eric Williams2013-03-281-2/+3
| | |
| * | Added landscape plugin, which gathers logs and configuration files for the ↵Eric Williams2013-03-281-0/+71
| |/ | | | | | | Ubuntu Advantage landscape client.
* | Merge pull request #109 from battlemidget/patch-1Bryn M. Reeves2013-04-031-6/+6
|\ \ | | | | | | Fix super call to proper general plugin name
| * | Fix super call to proper general plugin nameAdam Stokes2013-03-281-6/+6
| |/ | | | | | | | | | | | | | | 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>
* | Merge pull request #107 from battlemidget/patch-1Bryn M. Reeves2013-04-031-1/+1
|\ \ | |/ |/| Only specify the var/log/juju dir
| * Only specify the var/log/juju dirAdam Stokes2013-03-281-1/+1
|/ | | Only the juju directory exists in /var/log/ so grab that rather than globbing for the files.
* Merge pull request #103 from battlemidget/patch-1Bryn M. Reeves2013-03-2614-21/+93
|\ | | | | Port several plugins to Debian/Ubuntu
| * Update man page date to reflect recent changes to the informationAdam Stokes2013-03-251-1/+1
| | | | | | | | Signed-off-by: Adam Stokes <hackr@cypherbook.com>
| * Merge git://github.com/sosreport/sosreport into patch-1Adam Stokes2013-03-253-48/+89
| |\ | |/ |/|
* | Clean up command line option handlingBryn M. Reeves2013-03-251-12/+12
| | | | | | | | | | | | | | | | 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>
* | Man page updatesBryn M. Reeves2013-03-251-26/+64
| |
* | Rename --silent to --quietBryn M. Reeves2013-03-253-8/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '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>
* | Improve UI text readabilityBryn M. Reeves2013-03-251-2/+2
| | | | | | | | Indent paragraphs in the UI text to improve readability.
| * Conforming naming scheme in a few pluginsAdam Stokes2013-03-256-11/+13
| | | | | | | | | | | | | | 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>
| * Merge branch 'patch-1' of github.com:battlemidget/sosreport into patch-1Adam Stokes2013-03-251-1/+0
| |\
| | * Update __init__.pyAdam Stokes2013-03-251-1/+0
| | | | | | | | | Remove debugging pprint
| * | Inherit system within DebianSystem pluginAdam Stokes2013-03-251-1/+1
| |/ | | | | | | Signed-off-by: Adam Stokes <hackr@cypherbook.com>
| * Fixes #104Adam Stokes2013-03-251-1/+3
| | | | | | | | Signed-off-by: Adam Stokes <hackr@cypherbook.com>
| * Port several plugins to Debian/UbuntuAdam Stokes2013-03-2510-15/+84
|/ | | | | | | | [ Louis Bouchard ] - Ported several plugins to work across Red Hat, Debian and Ubuntu. Signed-off-by: Adam Stokes <hackr@cypherbook.com>
* Merge pull request #102 from sosreport/bmr-template-text-fixesBryn M. Reeves2013-03-255-29/+111
|\ | | | | Fix UI text templating
| * Fix Fedora policy detectionBryn M. Reeves2013-03-221-2/+3
| | | | | | | | | | | | | | | | 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>
| * Improve policy template handlingBryn M. Reeves2013-03-225-29/+110
|/ | | | | | | | | | | | | | | | | 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>
* Merge pull request #101 from battlemidget/cloud-pluginsBryn M. Reeves2013-03-213-0/+82
|\ | | | | | | | | | | | | | | | | Add modules for Ubuntu cloud infrastructure - Azure hypervisor support - MAAS - Juju
| * Merge git://github.com/sosreport/sosreport into cloud-pluginsAdam Stokes2013-03-181-0/+1
| |\ | |/ |/|
* | Merge pull request #98 from battlemidget/patch-1Bryn M. Reeves2013-03-181-0/+1
|\ \ | | | | | | Line 243 attempts to access non-imported regex
| * | Line 243 attempts to access non-imported regexAdam Stokes2013-02-111-0/+1
| | | | | | | | | This was a result from existing unittests.