aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix traceback when sar module is force-enabledBryn M. Reeves2013-11-071-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | Forward port of the following sos-2.2 fix: commit d4e3bcc3ed247d8d9a1228c3661b6456f534daf4 Author: Bryn M. Reeves <bmr@redhat.com> Date: Mon Aug 12 18:42:48 2013 +0100 Fix traceback when sar module is force-enabled The sar plug-in assumes that when it runs the /var/lib/sa directory and binaries exist (they are part of checkenabled() for this module). This leads to a traceback when the plug-in lists a non-existent directory on hosts where sar is not installed but '-o sar' is specified on the command line. Catch the exception and exit gracefully. (Edited by bmr to improve error message logged). Signed-off-by: David Kutálek <dkutalek@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Fix foreman and katello for sos-3.xBryn M. Reeves2013-11-072-26/+16
| | | | | | | | | | | The foreman and katello plug-ins committed in 4650d3f use the sos-2.2 APIs and class names. This breaks sos since the plug-ins will not load under 3.0. Fix up the modules to use the new names and API conventions and ensure that they load correctly. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Restore --build command line optionBryn M. Reeves2013-11-073-26/+51
| | | | | | | | | | | | | | | Releases of sos prior to 3.0 included the '--build' option to disable creation of a compressed archive and to leave the temporary build tree in place. This was removed with the reorganisation of the archive classes to support in-line archiving. Since all supported policies are now using an archive that derives from FileCacheArchive (commit 0178d5f) the option can be re-introduced. Archive classes that do not accumulate files in a temporary directory will return the path to the in-progress archive file. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Normalize temporary directory pathBryn M. Reeves2013-11-071-1/+2
| | | | | | | | | | | | | The archive and temporary file handling classes expect to operate on absolute paths. If a user specifies a relative path via the command line --tmp-dir switch it must be normalized before being passed to these classes to provide correct behaviour. Failing to do this causes a variety of errors: the final archive cannot be created since the path is constructed incorrectly and temporary files are not cleaned up since they are not at the "expected" location. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Add domainname output to the NIS plug-inBryn M. Reeves2013-11-051-0/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* maas: Dont capture everything /var/lib/maasAdam Stokes2013-11-051-1/+1
| | | | | | | | ephermeral images and other pickled objects are stored here which can increase the sosreport archive to well over 2G. so only capture the readable files which is only the dhcp leases and dhcp interface file. Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Added ubuntu/debian specific grub.cfg pathLouis Bouchard2013-11-041-0/+1
| | | | | | | Fixes: #214 Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Add a few more contributors to AUTHORSAdam Stokes2013-11-011-0/+3
| | | | | | Guy, Flavio, and Jeremy Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Generate report by defaultAdam Stokes2013-11-011-4/+4
| | | | | | Fixes #2 Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Do not copy ceph keyring filesAdam Stokes2013-11-011-0/+3
| | | | | | Fixes #200 Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Merge branch 'add-neutron-check-enabled' of github.com:jdutton/sosreport ↵Adam Stokes2013-10-311-0/+6
|\ | | | | | | | | | | | | | | into jdutton-add-neutron-check-enabled Fixes #211 Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * Add check_enabled for neutron pluginJeff Dutton2013-10-311-0/+6
| | | | | | | | Signed-off-by: Jeff Dutton <jeff.r.dutton@gmail.com>
* | Include Brent Eagles into authors fileAdam Stokes2013-10-311-0/+1
|/ | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Scrub credentials from openssl pluginAdam Stokes2013-10-301-0/+11
| | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Add IBM contributors to AUTHORSAdam Stokes2013-10-301-0/+3
| | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Change add_copy_spec calls that are passed array to use add_copy_specsJeff Dutton2013-10-303-2/+3
| | | | | Signed-off-by: Jeff Dutton <jeff.r.dutton@gmail.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Fix config file regressionBryn M. Reeves2013-10-301-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | Changes in the organisation of sos in commit 6ea48cb broke the reading of custom configuration files (either the default /etc/sos.conf or one specified on the command line using --config-file): Traceback (most recent call last): File "/usr/sbin/sosreport", line 23, in <module> main(sys.argv[1:]) File "/usr/lib/python2.7/site-packages/sos/sosreport.py", line 1167, in main sos.execute() File "/usr/lib/python2.7/site-packages/sos/sosreport.py", line 1133, in execute self._set_tunables() File "/usr/lib/python2.7/site-packages/sos/sosreport.py", line 798, in _set_tunables if not opt.split('.')[0] in self.disabled: AttributeError: 'SoSReport' object has no attribute 'disabled' The code attempts to reference the obsolete 'disabled' list of plug-ins (rather than calling self._get_disabled_plugins()) and failed to initialise the plugopts list to an empty deque. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* This patch defines a new PowerPC Plugin to collect generic Power logs.Bryn M. Reeves2013-10-301-0/+73
| | | | | | | | | | | | | | | Based on further platform checks,IBM Power System specific logs and commands will be collected. This would help IBM Power system users to collect system data in one shot by running sosreport. Signed-off-by: Bharani C.V. <bharanve@linux.vnet.ibm.com> Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com> --- sos/plugins/powerpc.py | 73 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 73 insertions(+) create mode 100644 sos/plugins/powerpc.py
* Fix --alloptionsBryn M. Reeves2013-10-301-1/+2
| | | | | | | | | | During the re-organisation of sosreport.py the code to apply all boolean options to plug-ins was factored out into its own function, _set_all_options() however this was never called from the main execute() method. Fix this to allow the option to work as previously. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Check for usable temporary directoryBryn M. Reeves2013-10-301-0/+6
| | | | | | | | A user may override the sos temporary directory via --tmp-dir to a non-existent or unwritable location. Check for this before initialising temporary files and exit with an error message. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Remove version checks in gluster plug-inBryn M. Reeves2013-10-291-16/+2
| | | | | | | Current distributions do not include the legac package versions that the plug-in was checking for. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Limit default sar data collectionBryn M. Reeves2013-10-291-1/+11
| | | | | | | | | | | By default the general module will scoop up all files under the /var/log/sa directory. With some configurations this path could contain many GB of data. Use add_copy_spec_limit() by default for sar data and add a new option sar.all_sar=False to allow the user to override this. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Move sar data colletion to sar plug-inBryn M. Reeves2013-10-292-1/+1
| | | | | | | | | | | Historically the general plug-in collected the content of the /var/log/sa directory while a separate sar plug-in handles the text formatted summary files. This makes little sense and adds to the bloat in the "general" module; move the collection to the sar plug-in itself. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Remove capturing redundant database export in MAASAdam Stokes2013-10-251-1/+0
| | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Remove OpenStack Dashboard plug-inBryn M. Reeves2013-10-221-82/+0
| | | | | | | | | Following the reorganisation of the OpenStack plug-ins the dashboard module now duplicates functionality provided by the horizon plug-in. Remove it and keep all the horizon and dashboard support in a single module. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Make OpenStack Keystone password removal defaultBryn M. Reeves2013-10-221-16/+15
| | | | | | | | The keystone plug-in used a command line option to enable password removal. This should be the default behaviour - remove the option and always apply the password replacement rules. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Fix configuration files paths for novaXavier Queralt2013-10-221-3/+2
| | | | Signed-off-by: Xavier Queralt <xqueralt@redhat.com>
* Fix password protection for novaXavier Queralt2013-10-221-14/+11
| | | | Signed-off-by: Xavier Queralt <xqueralt@redhat.com>
* Split out Cinder, Dashboard, Nova and Quantum from OpenStackBryn M. Reeves2013-10-225-135/+357
| | | | | | | | Split out support for the OpenStack Cinder, Dashboard, Nova and Quantum components from the single OpenStack plug-in. Signed-off-by: Jeremy Agee <jagee@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Update AUTHORSAdam Stokes2013-10-211-0/+1
|
* Merge pull request #206 from jamesodhunt/masterAdam Stokes2013-10-212-0/+66
|\ | | | | | | Addition of plugins for procenv and upstart. Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * Added plugin for Upstart (event-based init daemon).James Hunt2013-10-211-0/+41
| | | | | | | | Signed-off-by: James Hunt <james.hunt@ubuntu.com>
| * Added plugin for procenv (process environment).James Hunt2013-10-211-0/+25
|/ | | | Signed-off-by: James Hunt <james.hunt@ubuntu.com>
* Update readme to point to contributor guidelinesAdam Stokes2013-10-181-3/+11
| | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Improve plugin by adding apt log collection and update AUTHORSAdam Stokes2013-10-182-1/+2
|\ | | | | | | | | | | | | | | Closes: #204 Added Lukas to AUTHORS file for his contribution of katello/foreman Signed-off-by: Louis Bouchard <louis.bouchard@ubuntu.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * Improve plugin by adding apt log collectionLouis Bouchard2013-10-161-1/+1
| | | | | | | | Closes: #204
* | Adding plugins foreman and katelloAdam Stokes2013-10-183-38/+72
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since CloudForms 1.0 has been renamed to Satellite 6.0, I am deleting this plugin and providing two other plugins. Both projects Katello and Foreman are upstream projects for Satellite 6.0 Red Hat products. Since both projects support deployment on non-redhat systems, they both deliver own reporting script that collects information from target system. Both scripts have options not to create tarball and to provide output to desired directory. This has been tested on RHEL 6.4. Signed-off-by: Lukas Zapletal <lzap+git@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
| * | Adding plugins foreman and katelloLukas Zapletal2013-10-143-38/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since CloudForms 1.0 has been renamed to Satellite 6.0, I am deleting this plugin and providing two other plugins. Both projects Katello and Foreman are upstream projects for Satellite 6.0 Red Hat products. Since both projects support deployment on non-redhat systems, they both deliver own reporting script that collects information from target system. Both scripts have options not to create tarball and to provide output to desired directory. This has been tested on RHEL 6.4. Signed-off-by: Lukas Zapletal <lzap+git@redhat.com>
* | | Fix shell output unittestsAdam Stokes2013-10-181-2/+2
| | | | | | | | | | | | | | | | | | Since fix for 0ed431a went in we revert our unittests to the old (working) behavior. Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* | | Move Quantum/Neutron collection into its own plugin.Brent Eagles2013-10-172-14/+173
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed quantum specific collection commands from the openstack plugin and put them into a separate module. The plugin is also neutron/quantum agnostic and should work for basic collection in grizzly and havana based OpenStack installs. Open vSwitch and network namespace collection also added. Conflicts: sos/plugins/openstack.py Signed-off-by: Brent Eagles <beagles@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* | | Verify fewer packages in rpm plug-inBryn M. Reeves2013-10-161-1/+4
| |/ |/| | | | | | | | | | | | | The current default set of packages to verify includes debuginfo and devel subpackages. Filter these out to reduce the time spent running rpm -V. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* | Add OpenStack Keystone and Horizon plug-insBryn M. Reeves2013-10-153-18/+149
| | | | | | | | | | | | | | | | Split out the Keystone and Horizon support from the main OpenStack plug-in and put them in their own modules. Signed-off-by: Jeremy Agee <jagee@redhat.com> Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* | Remove obsolete workaround for newline added by pipe.communicate()Bryn M. Reeves2013-10-151-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | Older versions of the python runtime would add a newline to output obtained via pipe.communicate(). Current versions do not leading to output being stored in reports without a trailing newline character. Remove the workaround to ensure that stored content matches that obtained via simple shell redirection ("/some/command > file"). Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* | Remove useless os.path.exists check for brctl executableBryn M. Reeves2013-10-151-6/+5
|/ | | | | | | | | | | The networking module previously checked that its hard-coded path for the brctl command existed before collecting output. This was missed in the conversion to using policy defined PATH search for external commands in commit e0d132e. Similar problems were fixed in commits 8b10cb0, 374da99, 8883155. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Fix exception in command output substitutionBryn M. Reeves2013-10-061-1/+4
| | | | | | | | | | | | | | | | If an attempt is made to apply command output substitution via Plugin.do_cmd_output_sub() and no output has been collected (i.e. called['file'] == None) an exception is thrown by os.path.join(). The exception is also not logged properly due to an attempt in the do_cmd_output_sub() exception handler to access an unbound local variable (path - the exception occurs before it is assigned). Fix both of these by checking for an empty file entry and avoiding access to the path variable from the exception handler. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Include /etc/yaboot.conf in boot plug-inBryn M. Reeves2013-10-041-0/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Do not exit on interactive mode read exceptionsBryn M. Reeves2013-10-031-1/+1
| | | | | | | | | | | Currently the default policy object will exit without logging any errors if an exception (e.g. EOF while reading line) is caught during reads for interactive mode prompts. Instead re-set the report name to a sane value (localname) and attempt to continue. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* No TTY should not imply batch modeBryn M. Reeves2013-10-031-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 6ea48cbb introduced an isatty() check on stdin and automatically sets batch mode if we're not attached to a TTY. Although the logic of this seems simple enough the combination of this and commit 4b46e04 breaks valid uses that have worked since sos-1.x, e.g.: Traceback (most recent call last): File "/usr/sbin/sosreport", line 23, in <module> main(sys.argv[1:]) File "/usr/lib/python2.7/site-packages/sos/sosreport.py", line 1171, in main sos.execute() File "/usr/lib/python2.7/site-packages/sos/sosreport.py", line 1133, in execute self._setup_logging() File "/usr/lib/python2.7/site-packages/sos/sosreport.py", line 630, in _setup_logging self.opts.batch = True File "/usr/lib/python2.7/site-packages/sos/sosreport.py", line 324, in batch self._check_options_initialized() File "/usr/lib/python2.7/site-packages/sos/sosreport.py", line 229, in _check_options_initialized + "from command line") ValueError: SoSOptions object already initialized from command line Given the actual semantics of --batch (do not prompt for or read user input from stdin) there is no need to enable it if stdin is a pipe or other non-TTY file descriptor. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* Merge pull request #202 from karibou/masterAdam Stokes2013-09-301-0/+31
|\ | | | | | | | | Add new plugin for apt Signed-off by: Adam Stokes <adam.stokes@ubuntu.com>
| * Add new plugin for aptLouis Bouchard2013-09-301-0/+31
|/ | | | Closes #201
generated by cgit (git 2.34.1) at 2024-11-12 15:01:46 +0000