diff options
| author | Bryn M. Reeves <bmr@redhat.com> | 2014-02-14 16:54:15 +0000 |
|---|---|---|
| committer | Bryn M. Reeves <bmr@redhat.com> | 2014-02-14 16:54:15 +0000 |
| commit | 2591ac0719c256af3ed3b392f5bc5972ed3a1104 (patch) | |
| tree | c89863c1c21c1472f245577caf524654a7d80781 | |
| parent | 61e8147e436533c5ccb75a6061a4fcc7368970b6 (diff) | |
| download | sos-2591ac0719c256af3ed3b392f5bc5972ed3a1104.tar.gz | |
Remove useless check_enabled() from sar plugin
The sar plugin implemented a check_enabled() to catch cases where
the plugin is force-enabled on a system with no sar installation.
This is better handled by runtime checking in setup() which the
plugin also does. Kill the check and its annoying error message.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
| -rw-r--r-- | sos/plugins/sar.py | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/sos/plugins/sar.py b/sos/plugins/sar.py index 10862083..cf53d5bb 100644 --- a/sos/plugins/sar.py +++ b/sos/plugins/sar.py @@ -28,14 +28,6 @@ class Sar(Plugin,): # size-limit SAR data collected by default (MB) sa_size = 20 - def check_enabled(self): - # check to see if we are force-enabled with no sar installation - if not os.path.exists(self.sa_path) or not os.path.isdir(self.sa_path): - self.soslog.info("sar directory %s does not exist" % self.sa_path - + " or is not a directory") - return False - return True - def setup(self): if self.get_option("all_sar"): self.sa_size = 0 |