diff options
| author | Valerii Ponomarov <vponomar@redhat.com> | 2019-03-12 16:40:28 +0530 |
|---|---|---|
| committer | Valerii Ponomarov <vponomar@redhat.com> | 2019-03-12 16:42:57 +0530 |
| commit | be08b504a8cc284c3feeae69a99fc7ec2fdbc3c0 (patch) | |
| tree | 3870e8d41b21ad3e24f3b1835a04d5801c6e4c3f /cns-libs/cnslibs | |
| parent | 9bcec5a15f8c52fb5615e469e31a01fe08523a29 (diff) | |
Fix error messages in wait_for_pod_be_ready and enable_pvc_resize funcs
"wait_for_pod_be_ready" and "enable_pvc_resize" hide errors details
when errors appear. So, fix it by printing full error info.
Change-Id: I250ff723c0453fd487f8c1810cdd2c1d7d2ec16f
Diffstat (limited to 'cns-libs/cnslibs')
| -rw-r--r-- | cns-libs/cnslibs/common/cns_libs.py | 4 | ||||
| -rw-r--r-- | cns-libs/cnslibs/common/openshift_ops.py | 3 |
2 files changed, 4 insertions, 3 deletions
diff --git a/cns-libs/cnslibs/common/cns_libs.py b/cns-libs/cnslibs/common/cns_libs.py index 03966475..3c7bdc91 100644 --- a/cns-libs/cnslibs/common/cns_libs.py +++ b/cns-libs/cnslibs/common/cns_libs.py @@ -127,8 +127,8 @@ def enable_pvc_resize(master_node): "/usr/local/bin/master-restart controllers") ret, out, err = g.run(master_node, cmd, "root") if ret != 0 or out == "": - err_msg = ("failed to execute cmd %s on %s, err %s" - % (cmd, master_node, out)) + err_msg = "Failed to execute cmd %s on %s\nout: %s\nerr: %s" % ( + cmd, master_node, out, err) g.log.error(err_msg) raise ExecutionError(err_msg) diff --git a/cns-libs/cnslibs/common/openshift_ops.py b/cns-libs/cnslibs/common/openshift_ops.py index b7390152..0d44df1f 100644 --- a/cns-libs/cnslibs/common/openshift_ops.py +++ b/cns-libs/cnslibs/common/openshift_ops.py @@ -930,7 +930,8 @@ def wait_for_pod_be_ready(hostname, pod_name, ":.status.phase") % pod_name ret, out, err = g.run(hostname, cmd, "root") if ret != 0: - msg = ("failed to execute cmd %s" % cmd) + msg = "Failed to execute cmd: %s\nout: %s\nerr: %s" % ( + cmd, out, err) g.log.error(msg) raise exceptions.ExecutionError(msg) output = out.strip().split() |