diff options
| author | Bryn M. Reeves <bmr@redhat.com> | 2017-04-19 14:25:50 +0100 |
|---|---|---|
| committer | Bryn M. Reeves <bmr@redhat.com> | 2017-04-19 14:25:50 +0100 |
| commit | 5d76e3b8ce0a983c511c0a50e67cf8f7886914f0 (patch) | |
| tree | 6b1c3d13212c513fdcd3bcd9e80c8123f1e1f7d7 | |
| parent | a2c387ae1687170e827e0aa2b49faf759c015798 (diff) | |
| download | sos-5d76e3b8ce0a983c511c0a50e67cf8f7886914f0.tar.gz | |
[Plugin] remove bogus checks from get_cmd_output_now()
The underlying get_command_output() function already checks the
return status of the command: checking it again in the _now()
version is useless.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
| -rw-r--r-- | sos/plugins/__init__.py | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/sos/plugins/__init__.py b/sos/plugins/__init__.py index 35edcb1c..49da6f26 100644 --- a/sos/plugins/__init__.py +++ b/sos/plugins/__init__.py @@ -688,10 +688,6 @@ class Plugin(object): result = self.get_command_output(exe, timeout=timeout, stderr=stderr, chroot=chroot, runat=runat, env=env) - # 126 means 'found but not executable' - if result['status'] == 126 or result['status'] == 127: - return None - self._log_debug("collected output of '%s' in %s" % (exe.split()[0], time() - start)) |