| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
Fixes: #609.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
Resolves #551
Closes #568
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
Closes #578
Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
| |
Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
|
|
|
|
|
|
|
|
| |
Resolves #553
Closes #569
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
| |
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
|
|
|
| |
Expand the test coverage for Python 3.5 and
nightly build runtimes.
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
Utilize their container based build system for faster runs. Update
test version matrix.
Signed-off-by: Adam Stokes <adam.stokes@ubuntu.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
"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
|
|
|
|
|
|
|
|
|
| |
server_auth secrets in /etc/neutron/plugins/ml2/restproxy.ini need to be
obfuscated.
Resolves: #639
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
Call methods from the self class within "self." scope.
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
| |
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- 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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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 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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
This is useful information to be correlated with shutdown/reboot
events in /var/log/messages to identify graceful shutdowns/reboots
and lower false positives about system crashes.
Resolves #572.
Signed-off-by: Alexandru Juncu <alexj@linux.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
| |
Resolves #580.
Signed-off-by: Peter Portante <peter.portante@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
| |
When SYSROOT is not '/' relative symlinks need to be trimmed to
remove the extra leading '../' returned by
abspath('/sysroot/...').
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
| |
The abspath() call in _copy_symlink returns a host-relative path
(when SYSROOT is not '/'). Pass this directly to _do_copy_path()
without stripping the SYSROOT path component.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add journalctl output for the following kubernetes units:
kube-apiserver
kube-controller-manager
kube-scheduler
kube-proxy
Signed-off-by: Neependra Khare <nkhare@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
|