use the return code to determine if lsb_release is present

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

diff --git a/ceph_deploy/hosts/__init__.py b/ceph_deploy/hosts/__init__.py
index 96a976e14e52d2ff3a7ad792435d57766f317572..fb14b7727858310acd76ac9f69d3368a164f719d 100644 (file)
--- a/ceph_deploy/hosts/__init__.py
+++ b/ceph_deploy/hosts/__init__.py
@@ -67,8 +67,8 @@ def lsb_fallback(conn):
 
 
 def has_lsb(conn):
-    stdout, stderr, _ = wrappers.Popen(conn, logger, ['which', 'lsb_release'])
-    return stderr == ''
+    _, _, ret_code = wrappers.Popen(conn, logger, ['which', 'lsb_release'])
+    return ret_code == 0
 
 
 def _get_distro(distro, fallback=None):