From 7244b7cef18152e5bac2176ff8f94828eca6fbf2 Mon Sep 17 00:00:00 2001
From: Sage Weil <sage@redhat.com>
Date: Wed, 22 Jan 2020 16:37:51 -0600
Subject: [PATCH] mgr/cephadm: base upgrade check on return code, not stderr

Signed-off-by: Sage Weil <sage@redhat.com>
---
 src/pybind/mgr/cephadm/module.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pybind/mgr/cephadm/module.py b/src/pybind/mgr/cephadm/module.py
index cccb7756cddc3..d64701b71bda9 100644
--- a/src/pybind/mgr/cephadm/module.py
+++ b/src/pybind/mgr/cephadm/module.py
@@ -395,7 +395,7 @@ class CephadmOrchestrator(MgrModule, orchestrator.OrchestratorClientMixin):
             })
             if not self.upgrade_state or self.upgrade_state.get('paused'):
                 return False
-            if err:
+            if ret:
                 self.log.info('Upgrade: not safe to stop %s.%s' %
                               (s.service_type, s.service_instance))
                 time.sleep(15)
-- 
2.39.5