Fix Dash FlexCounter to skip invalid ENI meter class 0

mukeshmv · mukeshmv · commit 2cdb9ee01eb7 · 2025-10-20T16:41:25.000Z
Signed-off-by: mukeshmv &lt;mukesh@pensando.io&gt;
diff --git a/syncd/FlexCounter.cpp b/syncd/FlexCounter.cpp
@@ -2031,7 +2031,9 @@ class DashMeterCounterContext : public BaseCounterContext
         sai_object_key_t object_key;
         object_key.key.meter_bucket_entry.eni_id = rid;
         object_key.key.meter_bucket_entry.switch_id = m_switchId;
-        for (uint32_t i = 0; i < m_meterBucketsPerEni; ++i) {
+        // Bulk fetch stats for ENI meter classes/buckets 1 - max capable.
+        // 0 is not a valid meter class and is used to represent traffic that is not metered
+        for (uint32_t i = 1; i <= m_meterBucketsPerEni; ++i) {
             object_key.key.meter_bucket_entry.meter_class = i;
             ctx.object_keys.push_back(object_key);
         }
diff --git a/tests/aspell.en.pws b/tests/aspell.en.pws
@@ -30,6 +30,7 @@ DPU
 EAPOL
 ECN
 EIO
+ENI
 ETERM
 ecmp
 ECMP

-Original file line number
+Diff line change
 EAPOL
 ECN
 EIO
 +ENI
 ETERM
 ecmp
 ECMP