From 42e6aaaccdbd802565ca1b323fbac272f4a5c5e4 Mon Sep 17 00:00:00 2001 From: Valerii Ponomarov Date: Tue, 26 Feb 2019 18:54:12 +0530 Subject: Fix error message in 'cmd_run_on_gluster_pod_or_node' function In case of error on CRS node we get incorrect error message. Instead of command error we get another kind of an error. Change-Id: Ibfd21d518c220951a981f23daa5d32b1963061f2 --- cns-libs/cnslibs/common/openshift_ops.py | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/cns-libs/cnslibs/common/openshift_ops.py b/cns-libs/cnslibs/common/openshift_ops.py index dd5f8e17..2899d531 100644 --- a/cns-libs/cnslibs/common/openshift_ops.py +++ b/cns-libs/cnslibs/common/openshift_ops.py @@ -790,6 +790,7 @@ def cmd_run_on_gluster_pod_or_node(ocp_client_node, cmd, gluster_node=None): """ # Containerized Glusterfs gluster_pods = oc_get_pods(ocp_client_node, selector="glusterfs-node=pod") + err_msg = "" if gluster_pods: if gluster_node: for pod_name, pod_data in gluster_pods.items(): @@ -803,7 +804,6 @@ def cmd_run_on_gluster_pod_or_node(ocp_client_node, cmd, gluster_node=None): else: gluster_pod_names = gluster_pods.keys() - err_msg = "" for gluster_pod_name in gluster_pod_names: try: pod_cmd = "oc exec %s -- %s" % (gluster_pod_name, cmd) @@ -824,12 +824,15 @@ def cmd_run_on_gluster_pod_or_node(ocp_client_node, cmd, gluster_node=None): try: return command.cmd_run(cmd, hostname=g_host) except Exception as e: - g.log.error( - "Failed to run '%s' command on '%s' Gluster node. " - "Error: %s" % (cmd, g_host, e)) - - raise exceptions.ExecutionError( - "Haven't found neither Gluster PODs nor Gluster nodes.") + err = ("Failed to run '%s' command on '%s' Gluster node. " + "Error: %s\n" % (cmd, g_host, e)) + err_msg += err + g.log.error(err) + + if not err_msg: + raise exceptions.ExecutionError( + "Haven't found neither Gluster PODs nor Gluster nodes.") + raise exceptions.ExecutionError(err_msg) def get_gluster_vol_info_by_pvc_name(ocp_node, pvc_name): -- cgit