cli: append error cause when graph or query executor is canceled by user
authorPhilippe Proulx <eeppeliteloop@gmail.com>
Tue, 9 Jul 2019 16:55:58 +0000 (12:55 -0400)
committerPhilippe Proulx <eeppeliteloop@gmail.com>
Tue, 9 Jul 2019 18:39:28 +0000 (14:39 -0400)
Otherwise the CLI prints "unknown error" at the end. Now it prints:

    ERROR:    [Babeltrace CLI] (babeltrace2.c:2579)
      Graph was canceled by user.

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>
Change-Id: Ia910e91d75586e30bb017bd9332615891606c115
Reviewed-on: https://review.lttng.org/c/babeltrace/+/1659
Tested-by: jenkins <jenkins@lttng.org>
src/cli/babeltrace2.c

index fa37cf18f8db30f932fd76d1ba3c383bbb2bff0a..fd8a89cd630c3a59efe1a5eb14d1e54fe8c57797 100644 (file)
@@ -173,7 +173,8 @@ int query(struct bt_config *cfg, const bt_component_class *comp_cls,
        }
 
        if (canceled) {
-               BT_LOGI("Canceled by user before executing the query: "
+               BT_CLI_LOGW_APPEND_CAUSE(
+                       "Canceled by user before executing the query: "
                        "comp-cls-addr=%p, comp-cls-name=\"%s\", "
                        "query-obj=\"%s\"", comp_cls,
                        bt_component_class_get_name(comp_cls), obj);
@@ -198,7 +199,8 @@ int query(struct bt_config *cfg, const bt_component_class *comp_cls,
 
                        if (usleep(sleep_time_us)) {
                                if (bt_query_executor_is_canceled(the_query_executor)) {
-                                       BT_LOGI("Query was canceled by user: "
+                                       BT_CLI_LOGW_APPEND_CAUSE(
+                                               "Query was canceled by user: "
                                                "comp-cls-addr=%p, comp-cls-name=\"%s\", "
                                                "query-obj=\"%s\"", comp_cls,
                                                bt_component_class_get_name(comp_cls),
@@ -1793,7 +1795,7 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component(
                case BT_GRAPH_CONNECT_PORTS_STATUS_OK:
                        break;
                case BT_GRAPH_CONNECT_PORTS_STATUS_CANCELED:
-                       BT_LOGI_STR("Graph was canceled by user.");
+                       BT_CLI_LOGW_APPEND_CAUSE("Graph was canceled by user.");
                        break;
                default:
                        BT_CLI_LOGE_APPEND_CAUSE(
@@ -2520,7 +2522,8 @@ int cmd_run(struct bt_config *cfg)
        }
 
        if (canceled) {
-               BT_LOGI_STR("Canceled by user before creating components.");
+               BT_CLI_LOGW_APPEND_CAUSE(
+                       "Canceled by user before creating components.");
                goto error;
        }
 
@@ -2533,7 +2536,8 @@ int cmd_run(struct bt_config *cfg)
        }
 
        if (canceled) {
-               BT_LOGI_STR("Canceled by user before connecting components.");
+               BT_CLI_LOGW_APPEND_CAUSE(
+                       "Canceled by user before connecting components.");
                goto error;
        }
 
@@ -2547,7 +2551,8 @@ int cmd_run(struct bt_config *cfg)
        }
 
        if (canceled) {
-               BT_LOGI_STR("Canceled by user before running the graph.");
+               BT_CLI_LOGW_APPEND_CAUSE(
+                       "Canceled by user before running the graph.");
                goto error;
        }
 
@@ -2571,11 +2576,12 @@ int cmd_run(struct bt_config *cfg)
                case BT_GRAPH_RUN_STATUS_OK:
                        break;
                case BT_GRAPH_RUN_STATUS_CANCELED:
-                       BT_LOGI_STR("Graph was canceled by user.");
+                       BT_CLI_LOGW_APPEND_CAUSE("Graph was canceled by user.");
                        goto error;
                case BT_GRAPH_RUN_STATUS_AGAIN:
                        if (bt_graph_is_canceled(ctx.graph)) {
-                               BT_LOGI_STR("Graph was canceled by user.");
+                               BT_CLI_LOGW_APPEND_CAUSE(
+                                       "Graph was canceled by user.");
                                goto error;
                        }
 
@@ -2586,7 +2592,8 @@ int cmd_run(struct bt_config *cfg)
 
                                if (usleep(cfg->cmd_data.run.retry_duration_us)) {
                                        if (bt_graph_is_canceled(ctx.graph)) {
-                                               BT_LOGI_STR("Graph was canceled by user.");
+                                               BT_CLI_LOGW_APPEND_CAUSE(
+                                                       "Graph was canceled by user.");
                                                goto error;
                                        }
                                }
This page took 0.030871 seconds and 4 git commands to generate.