|
|
7711bd |
From 0c83c8245ffab2ff3f568078651553f59ef475bb Mon Sep 17 00:00:00 2001
|
|
|
7711bd |
From: Martin Perina <mperina@redhat.com>
|
|
|
7711bd |
Date: Wed, 21 Aug 2019 11:17:44 +0200
|
|
|
7711bd |
Subject: [PATCH] [vdsm]: Fix executing shell commands
|
|
|
7711bd |
|
|
|
7711bd |
During review process of [1] it was suggested to replace '/bin/sh' with
|
|
|
7711bd |
just 'sh', but unfortunately this change caused an error to not include
|
|
|
7711bd |
information about VDSM mount points as described in [2]. This fix
|
|
|
7711bd |
restores previous state and add full path to shell executions.
|
|
|
7711bd |
|
|
|
7711bd |
[1] https://github.com/sosreport/sos/pull/1205
|
|
|
7711bd |
[2] https://bugzilla.redhat.com/1743304
|
|
|
7711bd |
|
|
|
7711bd |
Resolves: #1761
|
|
|
7711bd |
|
|
|
7711bd |
Signed-off-by: Martin Perina <mperina@redhat.com>
|
|
|
7711bd |
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
|
|
|
7711bd |
---
|
|
|
7711bd |
sos/plugins/vdsm.py | 4 ++--
|
|
|
7711bd |
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
|
7711bd |
|
|
|
7711bd |
diff --git a/sos/plugins/vdsm.py b/sos/plugins/vdsm.py
|
|
|
7711bd |
index 35eec5093..c4e2af5f6 100644
|
|
|
7711bd |
--- a/sos/plugins/vdsm.py
|
|
|
7711bd |
+++ b/sos/plugins/vdsm.py
|
|
|
7711bd |
@@ -93,8 +93,8 @@ def setup(self):
|
|
|
7711bd |
])
|
|
|
7711bd |
self.add_cmd_output([
|
|
|
7711bd |
"ls -ldZ /etc/vdsm",
|
|
|
7711bd |
- "su vdsm -s sh -c 'tree -l /rhev/data-center'",
|
|
|
7711bd |
- "su vdsm -s sh -c 'ls -lR /rhev/data-center'"
|
|
|
7711bd |
+ "su vdsm -s /bin/sh -c 'tree -l /rhev/data-center'",
|
|
|
7711bd |
+ "su vdsm -s /bin/sh -c 'ls -lR /rhev/data-center'"
|
|
|
7711bd |
])
|
|
|
7711bd |
self.add_cmd_output([
|
|
|
7711bd |
"lvm vgs -v -o +tags --config \'%s\'" % LVM_CONFIG,
|