Sandro Bonazzola has uploaded a new change for review.

Change subject: sos2: fix callExtProg usage
......................................................................

sos2: fix callExtProg usage

fix usage of callExtProg method causing sos plugin
to exit due to a TypeError exception.

Change-Id: I02574b2d4ac35bac137db82060ccc18b5d27a372
Bug-Url: https://bugzilla.redhat.com/1123808
Signed-off-by: Sandro Bonazzola <[email protected]>
(cherry picked from commit f732b107c58b5f732a6ce23f854a9c283f687228)
(cherry picked from commit efe47eedf3feb4e25a0fe487203c6f3ba8e81bb6)
---
M src/sos/plugins/ovirt.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-log-collector 
refs/changes/49/31349/1

diff --git a/src/sos/plugins/ovirt.py b/src/sos/plugins/ovirt.py
index ff0cc25..d754cb3 100644
--- a/src/sos/plugins/ovirt.py
+++ b/src/sos/plugins/ovirt.py
@@ -68,7 +68,7 @@
         if self.getOption('jbosstrace'):
             engine_pattern = "^ovirt-engine\ -server.*jboss-modules.jar"
             pgrep = "pgrep -f '%s'" % engine_pattern
-            lines = self.callExtProg(pgrep)['output'].splitlines()
+            lines = self.callExtProg(pgrep)[1].splitlines()
             engine_pids = [int(x) for x in lines]
             if not engine_pids:
                 self.soslog.error('Unable to get ovirt-engine pid')


-- 
To view, visit http://gerrit.ovirt.org/31349
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I02574b2d4ac35bac137db82060ccc18b5d27a372
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: ovirt-log-collector-3.4
Gerrit-Owner: Sandro Bonazzola <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to