summaryrefslogtreecommitdiffstats
path: root/roles
diff options
context:
space:
mode:
authorRodolfo Carvalho <rhcarvalho@gmail.com>2017-06-09 14:25:30 +0200
committerGitHub <noreply@github.com>2017-06-09 14:25:30 +0200
commit670e93904c7fe35dafd07148a83cd4ccaf43953d (patch)
tree006bab8b593a7f8b0a481bbc1cf3fb735b87b07b /roles
parenta092dff53070c4eaa942dae36fc8742a7d53959d (diff)
downloadopenshift-670e93904c7fe35dafd07148a83cd4ccaf43953d.tar.gz
openshift-670e93904c7fe35dafd07148a83cd4ccaf43953d.tar.bz2
openshift-670e93904c7fe35dafd07148a83cd4ccaf43953d.tar.xz
openshift-670e93904c7fe35dafd07148a83cd4ccaf43953d.zip
Improve code readability
Diffstat (limited to 'roles')
-rw-r--r--roles/openshift_health_checker/openshift_checks/mixins.py4
1 files changed, 3 insertions, 1 deletions
diff --git a/roles/openshift_health_checker/openshift_checks/mixins.py b/roles/openshift_health_checker/openshift_checks/mixins.py
index 1181784ab..7f3d78cc4 100644
--- a/roles/openshift_health_checker/openshift_checks/mixins.py
+++ b/roles/openshift_health_checker/openshift_checks/mixins.py
@@ -50,4 +50,6 @@ class DockerHostMixin(object):
"Unable to install required packages on this host:\n"
" {deps}\n{msg}"
).format(deps=',\n '.join(self.dependencies), msg=msg)
- return msg, result.get("failed") or result.get("rc", 0) != 0, result.get("changed")
+ failed = result.get("failed", False) or result.get("rc", 0) != 0
+ changed = result.get("changed", False)
+ return msg, failed, changed