From 7a0042efe0b513c635d8e9bcfa24a3f8e0c69f44 Mon Sep 17 00:00:00 2001 From: KrisNey-MSFT Date: Wed, 3 Aug 2022 15:09:15 -0700 Subject: [PATCH 1/2] Update sdn-features-packet-transforms.md Creating PR for Counters conversation --- .../general/design/sdn-features-packet-transforms.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/documentation/general/design/sdn-features-packet-transforms.md b/documentation/general/design/sdn-features-packet-transforms.md index f6c213ef8..7f4deea17 100644 --- a/documentation/general/design/sdn-features-packet-transforms.md +++ b/documentation/general/design/sdn-features-packet-transforms.md @@ -371,7 +371,7 @@ thinking about how to add Metering- and reconcile this in the P4 model. | TotalPacket | Total packets to/from a VM. Exposed to customer; 2 counters, 1 per direction | ENI | | TotalBytes | Total bytes to/from a VM. Exposed to customer; 2 counters, 1 per direction | ENI | | TotalUnicastPacketForwarded | | ENI | -| TotalMulticastPacketsForwarded | | ENI | +| TotalMulticastPacketsForwarded - check on Multicast | | ENI | | TcpConnectionsResetHalfTTL | TCP connections that had a TCP reset and its TTL cut down to 5 seconds | ENI | | NonSynStateful | Non-SYN TCP packets that are natted and not dropped by setting (SLB scenario) | ENI | | NumberOfFlowResimulated DuringPortTimer | Number of connections updated in an internal port-level update | ENI | @@ -435,13 +435,14 @@ thinking about how to add Metering- and reconcile this in the P4 model. | CreationRateMaxOtherFlow | follow above | ENI | | No ENI Match | evident | ENI | | CPS Counters | | ENI & Global | +| Pipeline Counters Ex: per Rule/ACL Match? | | ENI & Global | **Questions** - How often will we read? - What type of API to use? -- Will we push or pull from the Controller? +- Will we push or pull from the Controller? Could be a controller that streams (with a subscription - standard gNMI telemetry) for example. ## BGP From ede9ff05d7b516418a84022cc543061c980ef5f7 Mon Sep 17 00:00:00 2001 From: KrisNey-MSFT Date: Wed, 3 Aug 2022 15:38:20 -0700 Subject: [PATCH 2/2] Update sdn-features-packet-transforms.md --- documentation/general/design/sdn-features-packet-transforms.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/documentation/general/design/sdn-features-packet-transforms.md b/documentation/general/design/sdn-features-packet-transforms.md index 7f4deea17..b79333771 100644 --- a/documentation/general/design/sdn-features-packet-transforms.md +++ b/documentation/general/design/sdn-features-packet-transforms.md @@ -326,7 +326,7 @@ Action| VNI | ## Telemetry -## Counters +## Counters - need to ensure the proposed counters are supportable in the P4 behavioral model (DASH pipeline) Counters are objects for counting data per ENI. The following are their main characteristics: