diff options
| author |   Ole Troan <ot@cisco.com> | 2020-10-07 18:05:37 +0200 |
|---|---|---|
| committer |   Neale Ranns <nranns@cisco.com> | 2020-10-13 11:42:58 +0000 |
| commit | 148c7b768721231325a349fa82db693190513b53 (patch) | |
| tree | 94e0a9768eda3bc4923b1b3cc727dc637d877381 /src/plugins/map/ip4_map_t.c | |
| parent | e7c8396982607634b4c747870499671ffa53868e (diff) | |
stats: counters data model
This adds a new data model for counters.
Specifying the errors severity and unit.
A later patch will update vpp_get_stats to take advantage of this.
Only the map plugin is updates as an example.
New .api language:
A new "counters" keyword to define counter sets.
counters map {
none {
severity info;
type counter64;
units "packets";
description "valid MAP packets";
};
bad_protocol {
severity error;
type counter64;
units "packets";
description "bad protocol";
};
};
Each counter has 4 keywords. severity, which is one of error, info or warn.
A type, which is one of counter64 or gauge64.
units, which is a text field using units from YANG.
paths {
"/err/ip4-map" "map";
"/err/ip6-map" "map";
"/err/ip4-t-map" "map";
"/err/ip6-t-map" "map";
};
A new paths keyword that maps the counter-set to a path in the stats segment KV store.
Updated VPP CLI to include severity so user can see error counter severity.
DBGvpp# show errors
Count Node Reason Severity
13 ethernet-input no error error
Type: feature
Signed-off-by: Ole Troan <ot@cisco.com>
Change-Id: Ib2177543f49d4c3aef4d7fa72476cff2068f7771
Signed-off-by: Ole Troan <ot@cisco.com>
Diffstat (limited to 'src/plugins/map/ip4_map_t.c')
| -rw-r--r-- | src/plugins/map/ip4_map_t.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/src/plugins/map/ip4_map_t.c b/src/plugins/map/ip4_map_t.c index 7d16d7aa455..8ae76f331f6 100644 --- a/src/plugins/map/ip4_map_t.c +++ b/src/plugins/map/ip4_map_t.c @@ -684,12 +684,6 @@ ip4_map_t (vlib_main_t * vm, vlib_node_runtime_t * node, vlib_frame_t * frame) return frame->n_vectors; } -static char *map_t_error_strings[] = { -#define _(sym,string) string, - foreach_map_error -#undef _ -}; - /* *INDENT-OFF* */ VNET_FEATURE_INIT (ip4_map_t_feature, static) = { .arc_name = "ip4-unicast", @@ -706,7 +700,7 @@ VLIB_REGISTER_NODE(ip4_map_t_fragmented_node) = { .type = VLIB_NODE_TYPE_INTERNAL, .n_errors = MAP_N_ERROR, - .error_strings = map_t_error_strings, + .error_counters = map_error_counters, .n_next_nodes = IP4_MAPT_FRAGMENTED_N_NEXT, .next_nodes = { @@ -727,7 +721,7 @@ VLIB_REGISTER_NODE(ip4_map_t_icmp_node) = { .type = VLIB_NODE_TYPE_INTERNAL, .n_errors = MAP_N_ERROR, - .error_strings = map_t_error_strings, + .error_counters = map_error_counters, .n_next_nodes = IP4_MAPT_ICMP_N_NEXT, .next_nodes = { @@ -748,7 +742,7 @@ VLIB_REGISTER_NODE(ip4_map_t_tcp_udp_node) = { .type = VLIB_NODE_TYPE_INTERNAL, .n_errors = MAP_N_ERROR, - .error_strings = map_t_error_strings, + .error_counters = map_error_counters, .n_next_nodes = IP4_MAPT_TCP_UDP_N_NEXT, .next_nodes = { @@ -769,7 +763,7 @@ VLIB_REGISTER_NODE(ip4_map_t_node) = { .type = VLIB_NODE_TYPE_INTERNAL, .n_errors = MAP_N_ERROR, - .error_strings = map_t_error_strings, + .error_counters = map_error_counters, .n_next_nodes = IP4_MAPT_N_NEXT, .next_nodes = { |