diff options
author | Jake Hunsaker <jhunsake@redhat.com> | 2017-07-16 02:59:37 -0400 |
---|---|---|
committer | Bryn M. Reeves <bmr@redhat.com> | 2018-03-28 11:58:13 +0100 |
commit | a05619cb550e3597e569e1e53b9bbaece731457b (patch) | |
tree | 3a2ea3ef4dc62bc7a329e4fbfd83be675e54c6e2 | |
parent | 4d996d4c91f4d3172109d27a81bd2b0ca4d3bfdf (diff) | |
download | sos-a05619cb550e3597e569e1e53b9bbaece731457b.tar.gz |
[xfs] Remove logprint and add xfs stats
Removes the logprint option which wasn't particularly useful for sos to
capture, reference Red Hat Bugzilla 1250035.
Adds collection of /prox/fs/xfs which has useful xfs statistics.
Resolves: #621
Closes: #1061
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
-rw-r--r-- | sos/plugins/xfs.py | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/sos/plugins/xfs.py b/sos/plugins/xfs.py index 9521ee6a..ee310dd2 100644 --- a/sos/plugins/xfs.py +++ b/sos/plugins/xfs.py @@ -23,8 +23,6 @@ class Xfs(Plugin, RedHatPlugin, DebianPlugin, UbuntuPlugin): plugin_name = 'xfs' profiles = ('storage',) - option_list = [("logprint", 'gathers the log information', 'slow', False)] - def setup(self): mounts = '/proc/mounts' ext_fs_regex = r"^(/dev/.+).+xfs\s+" @@ -33,10 +31,6 @@ class Xfs(Plugin, RedHatPlugin, DebianPlugin, UbuntuPlugin): parts = e.split(' ') self.add_cmd_output("xfs_info %s" % (parts[1])) - if self.get_option('logprint'): - for dev in zip(self.do_regex_find_all(ext_fs_regex, mounts)): - for e in dev: - parts = e.split(' ') - self.add_cmd_output("xfs_logprint -c %s" % (parts[0])) + self.add_copy_spec('/proc/fs/xfs') # vim: set et ts=4 sw=4 : |