From: Tatjana Dehler <tdehler@suse.com>
Date: Wed, 14 Sep 2022 14:36:16 +0000 (+0200)
Subject: cephadm: consider stdout to get container version
X-Git-Tag: v17.2.6~149^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=4d0afc42b442cbca4047eb52a0457a05a05f6212;p=ceph.git

cephadm: consider stdout to get container version

Fixes: https://tracker.ceph.com/issues/57558
Signed-off-by: Tatjana Dehler <tdehler@suse.com>
(cherry picked from commit d6156ede8b2d26b76403be39b62106ec2c4fb3b7)
---

diff --git a/src/cephadm/cephadm b/src/cephadm/cephadm
index 4e48b261e2d4..07f12e75c717 100755
--- a/src/cephadm/cephadm
+++ b/src/cephadm/cephadm
@@ -627,10 +627,11 @@ class Monitoring(object):
         cmd = daemon_type.replace('-', '_')
         code = -1
         err = ''
+        out = ''
         version = ''
         if daemon_type == 'alertmanager':
             for cmd in ['alertmanager', 'prometheus-alertmanager']:
-                _, err, code = call(ctx, [
+                out, err, code = call(ctx, [
                     ctx.container_engine.path, 'exec', container_id, cmd,
                     '--version'
                 ], verbosity=CallVerbosity.QUIET)
@@ -638,12 +639,14 @@ class Monitoring(object):
                     break
             cmd = 'alertmanager'  # reset cmd for version extraction
         else:
-            _, err, code = call(ctx, [
+            out, err, code = call(ctx, [
                 ctx.container_engine.path, 'exec', container_id, cmd, '--version'
             ], verbosity=CallVerbosity.QUIET)
-        if code == 0 and \
-                err.startswith('%s, version ' % cmd):
-            version = err.split(' ')[2]
+        if code == 0:
+            if err.startswith('%s, version ' % cmd):
+                version = err.split(' ')[2]
+            elif out.startswith('%s, version ' % cmd):
+                version = out.split(' ')[2]
         return version
 
 ##################################