aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [openstack_swift] Limit log collection to .log files by defaultLee Yarwood2015-10-151-3/+9
| | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #448 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_*] fix setup() regressions introduced in 1699eabdLee Yarwood2015-10-158-20/+20
| | | | | | | | | | 1699eabd partially renamed various OpenStack plugin classes but was not complete when it was pulled in leading to various openstack plugins failing during setup. Resolves: #655 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [kernel] Add collection of /sys/kernel/slabsLouis Bouchard2015-10-151-0/+1
| | | | | | | | | | Needed when investigating slab related issue to identify how slabs are merged together. ls -lt shows symlink of merged slabs. Closes: #653, #654. Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
* [qpid_dispatch] code cleanup and adding authorshipPavel Moravec2015-10-151-9/+8
| | | | | | | | | Merged identical code into one loop iterating over ["ssl-certificate", "ssl-key", "ssl-trustfile"]. Added authorship. Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [archive] Force decoding if content is bytesLouis Bouchard2015-10-151-0/+2
| | | | | | | | | | | | | | | When content is coming from the output of the tail() method, it is a bytes in python3 and a string in python2. Force the bytes value to be decoded otherwise the later write() function will fail and nothing will be written to the file. Conditional decoding is necessary as the strings in python3 do not have a decode method. Closes #586 Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
* [archive] Add test to highlight py3 specific errorLouis Bouchard2015-10-151-0/+14
| | | | | | | When add_string receives output from tail() it fails in py3 since tail() returns a byte object. Mimic this behavior. Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com>
* [openstack_sahara] capture logs and packages in split modePoornima2015-10-151-0/+4
| | | | | | | | | Adding two services openstack-sahara-api and openstack-sahara-engine with required "-api + -engine" RPM package. Fixes: #641. Signed-off-by: Poornima M. Kshirsagar pkshiras@redhat.com
* [qpid] Add port and SSL related optionsPavel Moravec2015-10-151-17/+37
| | | | | | | | | | | | | Four options added: - port for specifying non-default listening port (sometimes necessary also for SSL) - ssl-certificate and ssl-key for SSL - ssl as rather auxiliary option to force qpid-* tools to use SSL even without ssl-* options specified Resolves: #634, #635 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [qpid] Fix missing commas in add_cmd_outputPavel Moravec2015-10-151-2/+2
| | | | | | | | Commands "qpid-config exchanges" and "qpid-config queues" lack comma in the list. That causes the commands are not collected by sosreport and executed. Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [docker] collect journald logs for docker unitBryn M. Reeves2015-10-071-0/+1
| | | | | | Fixes: #609. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [kubernetes] do not reverse-sort journald logsBryn M. Reeves2015-10-071-5/+5
| | | | | | | | Drop the '-r' when obtaining logs from journald to be consistent with other plugins: users can sort log entries in whatever order they prefer on the analysis system. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack_nova] Limit log collection to .log files by defaultLee Yarwood2015-09-301-4/+8
| | | | | | | | | | | | Also introduce the global all_logs and log_size options within the plugin. Allowing users to control when all logs are collected and limit the overall size of the logs collected. Resolves #447 Closes #647 Signed-off-by: Lee Yarwood <lyarwood@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [openstack_horizon] Only use regex against configuration filesLee Yarwood2015-09-301-1/+3
| | | | | | | | | | | | Ignore files from plugins such as tuskar [1] that can also be found in the `/etc/openstack-dashboard/` directory. [1] https://github.com/openstack/tuskar-ui Closes #645 Signed-off-by: Lee Yarwood <lyarwood@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [qpid_dispatch] add new plugin for Qpid DispatchPavel Moravec2015-09-301-0/+62
| | | | | | | | | | | | | | New plugin to collect valuable information for Qpid Dispatch Collects /etc/qpid-dispatch/qdrouterd.conf and output of qdstat -[a|n|c|m]. Allows passing arguments for Dispatch listening port and SSL stuff. Resolves: #623 Closes #624 Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [monit] add new plugin for monit monitoringPablo Iranzo Gómez2015-09-301-0/+78
| | | | | | | | | | | | | Collect monit status, action logs and configuration files. Usernames (in addition to passwords and other secrets) are removed since they may contain information such as email addresses considered sensitive by users. Closes #573 Signed-off-by: Pablo Iranzo Gómez <Pablo.Iranzo@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [boot] Collect efibootmgr outputPavel Moravec2015-09-301-0/+3
| | | | | | | | Resolves #551 Closes #568 Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [sosreport] fix command-line report defaultsPavel Moravec2015-09-291-11/+11
| | | | | | | | | | | | | | | | | opts.report is True regardless of using --no-report option or not. That means the test "if not self.opts.report:" to decide generation of reports is evaluated to False every time and no report is generated any time. This commit renames opts.report and SoSOptions._report to noreport (to get rid of the variable name confusion) and updates default value accordingly. Resolves #625 Closes #644 Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [systemd] Collect timedatectl for timezonejbainbri2015-09-291-1/+2
| | | | | | | | | | There is currently no way to see the system timezone in text format on a systemd system (eg: "America/New_York"). timedatectl provides this. Closes #611 Signed-off-by: Jamie Bainbridge <jbainbri at redhat dot com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [networking] gather per-namespace ip and ethtool dataRobb Manes2015-09-291-2/+59
| | | | | | | | | | | | | | | | | | | | | As more applications are beginning to use namespaces, notably OpenStack and container-based platforms, capturing namespace data is becoming essential. This patch alters the networking plugin so that it is namespace aware, collecting commands such as: $ ip address show $ ip route $ iptables-save For each individual namespace, along with per-device ethtool information for any devices that exist in said namespaces. Closes #642 Signed-off-by: Robb Manes <rmanes@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [openstack] Add new controller and compute profiles.Lee Yarwood2015-09-2911-11/+11
| | | | | | | Closes #578 Signed-off-by: Lee Yarwood <lyarwood@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [openstack] Remove 'OpenStack' from distro class names.Lee Yarwood2015-09-298-18/+18
| | | | Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [iscsi] get iscsiadm -m node -P 1 instead of -P 3Pavel Moravec2015-09-291-1/+1
| | | | | | | | Resolves #553 Closes #569 Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [travis-ci] Remove deprecated --use-mirrorsAdam Stokes2015-09-291-1/+1
| | | | Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [travis-ci] support 3.5 and nightly buildsAdam Stokes2015-09-291-0/+6
| | | | | | | Expand the test coverage for Python 3.5 and nightly build runtimes. Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [atomic] Add plugin for Atomic HostJake Hunsaker2015-09-291-0/+44
| | | | | | | | | | | | | This plugin is for Atomic Host systems. By default it will capture host status and ostree configuration. With the 'info' option it will collect 'atomic info' for unique images. Closes #549 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [networking] iptables: check module version and do ipv6Germano Veit Michel2015-09-291-2/+15
| | | | | | | | | | | | | If we grep for "mangle", both ipv4 and ipv6 versions might come up. This patch ensure that the right version is considered. Also, do the same we do for ipv4, but for ipv6. Filter, Mangle, NAT and exact counters. Closes #630 Signed-off-by: Germano Veit Michel <germano@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [gluster] added new commandshari gowtham2015-09-291-3/+14
| | | | | | | | | | The new commands added to gluster in the recent release were missing. this commit is to add those commands to the plugin. Closes #633 Signed-off-by: hari gowtham <hgowtham@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* Update travis-ci build propertiesAdam Stokes2015-09-291-2/+1
| | | | | | | Utilize their container based build system for faster runs. Update test version matrix. Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [kubernetes] Change 'minions' to 'nodes'.Jake Hunsaker2015-09-291-1/+1
| | | | | | | | | | | | Kube no longer uses 'minions' and only accepts 'nodes' in kubectl. Previous versions of kubectl all accepted both 'nodes' and 'minions' however since v1.0, 'minions' has been deprecated. Closes #638 Signed-off-by: Jake Hunsaker <jhunsake@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [docs] add contributor guidelines link to patch docsBryn M. Reeves2015-09-291-0/+3
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [config] remove 'general' section from example sos.confBryn M. Reeves2015-09-291-7/+0
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [openstack] New openstack_instack plugin.Lee Yarwood2015-09-251-0/+75
| | | | | | | | | | | | | | | | | | | | | Add a new plugin for instack [1] and instack-undercloud [2]. These two components are currently used to drive the deployment of the undercloud in RDO Manager and RHEL OSP director environments. Instack itself is considered low level tooling and is in turn called by higher level user applications such as the RDO Manager Openstack client plugin [3]. This plugin also adds a new openstack_undercloud profile to be used to tag simillar Openstack plugins that are only found in the undercloud. [1] https://github.com/rdo-management/instack [2] https://github.com/rdo-management/instack-undercloud [3] https://github.com/rdo-management/python-rdomanager-oscplugin Resolves #620. Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [openstack_swift] fix regex issue in postproc methodPoornima2015-09-101-1/+1
| | | | | | | | | | | | | | | | | | | | "The regex currently used by openstack_swift is: "/etc/swift/*.conf*" This matches any filename beginning with 'con': "/etc/swift/con*" and causes the plugin to attempt regex substitution on several binary files in the directory causing errors: "substitution failed for '/etc/swift/container.builder'" "regex substitution failed for '/etc/swift/container.ring.gz'" Purposing a fix by changing the regex to match only intended configuration files (*.conf*) in this directory. Fixes #636. Signed-off-by: Poornima M. Kshirsagar pkshiras@redhat.com
* [openstack_neutron] obfuscate server_auth in restproxy.iniPavel Moravec2015-09-071-1/+1
| | | | | | | | | server_auth secrets in /etc/neutron/plugins/ml2/restproxy.ini need to be obfuscated. Resolves: #639 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [ovirt] obfuscate passwords in aaa profilesTim Speetjens2015-09-041-0/+10
| | | | | | | | | | | With aaa profiles configured in /etc/ovirt-engine/aaa, all .properties files in this directory may contain passwords in keys vars.password, pool.default.ssl.truststore.password and potentially pool.default.auth.simple.password Resolves #588 Signed-off-by: Tim Speetjens <tim.speetjens@redhat.com>
* [memory] collect swapon --show output in bytesBryn M. Reeves2015-08-271-1/+1
| | | | | | | | | | | | | | | | Default is human readable: $ swapon --show NAME TYPE SIZE USED PRIO /dev/dm-0 partition 1.5G 0B -1 Should be: $ swapon --show --bytes NAME TYPE SIZE USED PRIO /dev/dm-0 partition 1610608640 0 -1 Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [networking] brctl command is run when bridge kernel module is loaded onlyKeigo Noha2015-08-272-7/+19
| | | | | | | | | | | | | | | Previously, bridge information were collected even if bridge kernel module were not loaded. brctl command requires and loads bridge and related kernel modules to accomplish its purporse. This behaviour causes unexpected and unnecessary kernel module loading. sosreport should not change the system configuration. This patch will change this behaviour to run brctl command only if bridge module is loaded. Closes #616 Signed-off-by: Keigo Noha <knoha@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [openstack] New Ironic plugin.Lee Yarwood2015-08-271-0/+84
| | | | | | | | | | | Add a new plugin for Ironic, a bare metal provisioning service for openstack. Closes #577 [1] https://wiki.openstack.org/wiki/Ironic Signed-off-by: Lee Yarwood <lyarwood@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [sosreport] fix command-line report defaultsPavel Moravec2015-08-271-1/+1
| | | | | | | | | | Original commit a7ef3ca was accidentally reverted in cb3d265. This commit applies the a7ef3ca again. Fixes #625 Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [ldap] Fix plugin runtime failure on debian basedLouis Bouchard2015-08-261-2/+2
| | | | | | | | | | Plugin should call DebianPlugin class and missing comma in call. Closes #626 Closes #627 Signed-off-by: Louis Bouchard <louis.bouchard@canonical.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [landscape] Collect landscape statusBryan Quigley2015-08-171-0/+1
| | | | | | | | | | The lsctl command gives more details on the current status of landscape. Closes #622 Signed-off-by: Bryan Quigley <bryan.quigley@canonical.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [sapnw] call self methods properlyPavel Moravec2015-08-041-2/+2
| | | | | | Call methods from the self class within "self." scope. Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [sapnw] Add check if saphostctrl is not presentPavel Moravec2015-08-031-13/+17
| | | | | | | | | | | | Split listing&collecting instances and dbs from lengthy setup(). Break execution when "inst_out = self.get_cmd_output_now" returns None. db_out is already checked this way. Resolves: #614 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [logs] fix reference to missing 'rsyslog_conf' variableBryn M. Reeves2015-08-031-0/+1
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [Plugin] fix handling of symlinks in non-sysroot environmentsBryn M. Reeves2015-08-031-0/+2
| | | | Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
* [sapnw] uses a deprecated Sets modulePavel Moravec2015-07-291-2/+1
| | | | | | | | | Use built-in set class instead of deprecated Set for sidsunique Resolves: #608 Signed-off-by: Pavel Moravec <pmoravec@redhat.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [openstack] Ensure openstack passwords and secrets are obfuscated.Lee Yarwood2015-07-1711-48/+100
| | | | | | | | | | | | | - Add a postproc method to ceilometer, cinder, glance, hoirzon and swift. - Add missing keys to the remaining plugins. - Modify the regular expression used by all plugins to ignore commented out keys. - Modify all plugins to use do_path_regex_sub() to apply regex to all collected configuration files. Resolves: #574 Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
* [plugin] pass stderr through _collect_cmd_outputPavel Moravec2015-07-131-1/+1
| | | | | | | | | | | | Commit f06efd6 removed passing stderr in _collect_cmd_output to get_cmd_output_now. That prevents passing stderr=False in several scenarios. This fix adds the argument to be passed back. Resolves: #600 Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
* [juju] Update log collection to have limitsBryan Quigley2015-07-121-1/+21
| | | | | | | | | | | | | | | | | | | Juju can name files differently depending on the deployment so we need iterate over all the files and capture at least some of them all. At the same time the files can get quite big so we need to be able to limit their size. Now captures upstart/juju-db.log which is usually quite small. Juju local is usually used for testing so we just capture all-machines.log for that (format /var/log/juju-name-name/) Lastly we capture the ls of all of the key directories just in case we missed something. Closes #593 Signed-off-by: Bryan Quigley <bryan.quigley@canonical.com> Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
* [networking] Collect plotnetcfg output.Lee Yarwood2015-07-081-0/+1
| | | | | | | | | | | | | | | | The plotnetcfg tool [1] has recently been packaged for Fedora and provides a useful method for visualising complex host network toplogies. In the case of Openstack hosts this includes Openvswitch devices, flows etc. The resulting file can be used to generate a PDF visualising this with the following command : `cat plotnetcfg | dot -Tpdf > output.pdf` [1] https://github.com/jbenc/plotnetcfg Signed-off-by: Lee Yarwood <lyarwood@redhat.com>