diff options
author | Dave Wallace <dwallacelf@gmail.com> | 2018-02-13 16:14:06 -0500 |
---|---|---|
committer | Marco Varlese <marco.varlese@suse.de> | 2018-02-14 15:13:00 +0000 |
commit | 5ba5837b150acb7c827809b262eb43230030422e (patch) | |
tree | 9be469e58ecc45f653304606d3a76b8fe78a1e54 | |
parent | 6d2f214dab3984995bdbe03ad30487ac2d01b555 (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.c | 8 | ||||
-rw-r--r-- | test/framework.py | 2 |
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) |