summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDave Wallace <dwallacelf@gmail.com>2018-02-13 16:14:06 -0500
committerMarco Varlese <marco.varlese@suse.de>2018-02-14 15:13:00 +0000
commit5ba5837b150acb7c827809b262eb43230030422e (patch)
tree9be469e58ecc45f653304606d3a76b8fe78a1e54
parent6d2f214dab3984995bdbe03ad30487ac2d01b555 (diff)
make-test: use api custom-dump to reduce log size.
- Reduce log size by using "api trace custom-dump" instead of "api trace dump". - Fix custom-dump output of cli_inband_t api message to include cli command being executed. New output: DBGvpp# api trace custom-dump /tmp/test vl_api_memclnt_delete_t: index: 2 handle: 0x301d8e10 SCRIPT: memclnt_create name vpp_api_test SCRIPT: sw_interface_dump all SCRIPT: control_ping SCRIPT: exec show run Old output: DBGvpp# api trace dump /tmp/test ---------- trace 0 ----------- vl_api_memclnt_delete_t: index: 33554432 handle: 0x108e1d3000000000 ---------- trace 1 ----------- vl_api_memclnt_create_t: name: vpp_api_test input_queue: 0x808e1d3000000000 context: 0 ctx_quota: 0 ---------- trace 2 ----------- vl_api_sw_interface_dump_t: _vl_msg_id: 61 client_index: 33554432 context: 0 name_filter_valid: 0 ---------- trace 3 ----------- vl_api_control_ping_t: _vl_msg_id: 712 client_index: 33554432 context: 0 ---------- trace 4 ----------- vl_api_cli_inband_t: _vl_msg_id: 715 client_index: 33554432 context: 0 length: 9 Change-Id: If740c861649a3a59b8cc7a777c23c3cf94b8ff87 Signed-off-by: Dave Wallace <dwallacelf@gmail.com>
-rw-r--r--src/vpp/api/custom_dump.c8
-rw-r--r--test/framework.py2
2 files changed, 8 insertions, 2 deletions
diff --git a/src/vpp/api/custom_dump.c b/src/vpp/api/custom_dump.c
index 49fe2a8bd26..a51b6aa2dc6 100644
--- a/src/vpp/api/custom_dump.c
+++ b/src/vpp/api/custom_dump.c
@@ -1806,9 +1806,15 @@ static void *vl_api_cli_inband_t_print
(vl_api_cli_inband_t * mp, void *handle)
{
u8 *s;
+ u8 *cmd = 0;
+ u32 length = ntohl (mp->length);
- s = format (0, "SCRIPT: cli_inband ");
+ vec_validate (cmd, length);
+ clib_memcpy (cmd, mp->cmd, length);
+ s = format (0, "SCRIPT: exec %v ", cmd);
+
+ vec_free (cmd);
FINISH;
}
diff --git a/test/framework.py b/test/framework.py
index e2b4d7bd566..bf21c57d199 100644
--- a/test/framework.py
+++ b/test/framework.py
@@ -474,7 +474,7 @@ class VppTestCase(unittest.TestCase):
self.logger.info("Moving %s to %s\n" % (tmp_api_trace,
vpp_api_trace_log))
os.rename(tmp_api_trace, vpp_api_trace_log)
- self.logger.info(self.vapi.ppcli("api trace dump %s" %
+ self.logger.info(self.vapi.ppcli("api trace custom-dump %s" %
vpp_api_trace_log))
else:
self.registry.unregister_all(self.logger)