aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* 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>
* 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.
| | * 3 New pluginsAdam Stokes2013-03-063-0/+82
| |/ |/| | | | | | | | | | | | | - sos/plugins/azure: Microsoft Azure Agent - sos/plugins/juju: Ubuntu JUJU - sos/plugins/maas: Ubuntu MAAS Signed-off-by: Adam Stokes <hackr@cypherbook.com>
* | Merge pull request #99 from battlemidget/patch-2Bryn M. Reeves2013-02-221-2/+2
|\ \ | |/ |/| Include new-line in shell_out
| * Include new-line in shell_outAdam Stokes2013-02-111-2/+2
|/
* Make add_file() more robustBryn M. Reeves2012-12-181-9/+15
| | | | | | 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().
* Force LC_ALL=C for external commandsBryn M. Reeves2012-12-181-1/+6
| | | | | | | | | | 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.
* Disable archive recursion tests for TarFileArchiveBryn M. Reeves2012-12-181-10/+14
| | | | | | The TarFileArchive class currently doesn't allow recursive directory addition using the add_file() interface - disable those tests for now.
* Fix import statement in archives_test.pyBryn M. Reeves2012-12-181-1/+1
|
* Separate archive classes from utilities moduleBryn M. Reeves2012-12-185-244/+276
| | | | | | | | | | | | | | | 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.
* Rename command output collection interfacesBryn M. Reeves2012-12-1764-360/+360
| | | | | | | | | | | 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.
* Merge pull request #94 from battlemidget/featureJesse Jaggars2012-12-141-1/+1
|\ | | | | Make Ubuntu/Debian depend on python-selinux (Closed GH:#93)