From 148c7b768721231325a349fa82db693190513b53 Mon Sep 17 00:00:00 2001 From: Ole Troan Date: Wed, 7 Oct 2020 18:05:37 +0200 Subject: 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 Change-Id: Ib2177543f49d4c3aef4d7fa72476cff2068f7771 Signed-off-by: Ole Troan --- src/plugins/map/map.api | 114 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 114 insertions(+) (limited to 'src/plugins/map/map.api') diff --git a/src/plugins/map/map.api b/src/plugins/map/map.api index fa32978591c..dfe255bf349 100644 --- a/src/plugins/map/map.api +++ b/src/plugins/map/map.api @@ -349,3 +349,117 @@ define map_param_get_reply bool tc_copy; u8 tc_class; }; + +/* + * MAP Error counters/messages + */ +counters map { + none { + severity info; + type counter64; + units "packets"; + description "valid MAP packets"; + }; + bad_protocol { + severity error; + type counter64; + units "packets"; + description "bad protocol"; + }; + sec_check { + severity error; + type counter64; + units "packets"; + description "security check failed"; + }; + encap_sec_check { + severity error; + type counter64; + units "packets"; + description "encap security check failed"; + }; + decap_sec_check { + severity error; + type counter64; + units "packets"; + description "decap security check failed"; + }; + icmp { + severity error; + type counter64; + units "packets"; + description "unable to translate ICMP"; + }; + icmp_relay { + severity error; + type counter64; + units "packets"; + description "unable to relay ICMP"; + }; + unknown { + severity error; + type counter64; + units "packets"; + description "unknown"; + }; + no_binding { + severity error; + type counter64; + units "packets"; + description "no binding"; + }; + no_domain { + severity error; + type counter64; + units "packets"; + description "no domain"; + }; + fragmented { + severity error; + type counter64; + units "packets"; + description "packet is a fragment"; + }; + fragment_memory { + severity error; + type counter64; + units "packets"; + description "could not cache fragment"; + }; + fragment_malformed { + severity error; + type counter64; + units "packets"; + description "fragment has unexpected format"; + }; + fragment_dropped { + severity error; + type counter64; + units "packets"; + description "dropped cached fragment"; + }; + malformed { + severity error; + type counter64; + units "packets"; + description "malformed packet"; + }; + df_set { + severity error; + type counter64; + units "packets"; + description "can't fragment, DF set"; + }; + time_exceeded { + severity error; + type counter64; + units "packets"; + description "time exceeded"; + }; +}; +paths { + "/err/ip4-map" "map"; + "/err/ip6-map" "map"; + "/err/ip4-t-map" "map"; + "/err/ip6-t-map" "map"; +}; -- cgit 1.2.3-korg