summaryrefslogtreecommitdiffstats
path: root/roles/lib_openshift
diff options
context:
space:
mode:
authorKenny Woodson <kwoodson@redhat.com>2017-03-01 16:54:14 -0500
committerKenny Woodson <kwoodson@redhat.com>2017-03-01 16:54:14 -0500
commit8621904168b5e99cbf03158d888e6883e627e1e9 (patch)
treed712353a12bcbb1c4fb8706b1480da75c4d4eedf /roles/lib_openshift
parent00683cbb4b222f702b8367b8d259d8e7049d2a8e (diff)
downloadopenshift-8621904168b5e99cbf03158d888e6883e627e1e9.tar.gz
openshift-8621904168b5e99cbf03158d888e6883e627e1e9.tar.bz2
openshift-8621904168b5e99cbf03158d888e6883e627e1e9.tar.xz
openshift-8621904168b5e99cbf03158d888e6883e627e1e9.zip
Fixed error handling when oc adm ca create-server-cert fails. Fixed a logic error in secure.
Diffstat (limited to 'roles/lib_openshift')
-rw-r--r--roles/lib_openshift/library/oc_adm_ca_server_cert.py3
-rw-r--r--roles/lib_openshift/src/class/oc_adm_ca_server_cert.py3
2 files changed, 6 insertions, 0 deletions
diff --git a/roles/lib_openshift/library/oc_adm_ca_server_cert.py b/roles/lib_openshift/library/oc_adm_ca_server_cert.py
index 9390ea7c1..74ef57b4e 100644
--- a/roles/lib_openshift/library/oc_adm_ca_server_cert.py
+++ b/roles/lib_openshift/library/oc_adm_ca_server_cert.py
@@ -1492,6 +1492,9 @@ class CAServerCert(OpenShiftCLI):
api_rval = server_cert.create()
+ if api_rval['returncode'] != 0:
+ return {'Failed': True, 'msg': api_rval}
+
return {'changed': True, 'results': api_rval, 'state': state}
########
diff --git a/roles/lib_openshift/src/class/oc_adm_ca_server_cert.py b/roles/lib_openshift/src/class/oc_adm_ca_server_cert.py
index 6ed1f2f35..18c69f2fa 100644
--- a/roles/lib_openshift/src/class/oc_adm_ca_server_cert.py
+++ b/roles/lib_openshift/src/class/oc_adm_ca_server_cert.py
@@ -122,6 +122,9 @@ class CAServerCert(OpenShiftCLI):
api_rval = server_cert.create()
+ if api_rval['returncode'] != 0:
+ return {'Failed': True, 'msg': api_rval}
+
return {'changed': True, 'results': api_rval, 'state': state}
########