Skip to content

Comments

[action] [PR:1725] Fix dash meter COUNTERS_DB keys to use VID instead of RID#1750

Merged
prsunny merged 3 commits intosonic-net:202511from
mssonicbld:cherry/202511/1725
Feb 20, 2026
Merged

[action] [PR:1725] Fix dash meter COUNTERS_DB keys to use VID instead of RID#1750
prsunny merged 3 commits intosonic-net:202511from
mssonicbld:cherry/202511/1725

Conversation

@mssonicbld
Copy link
Collaborator

Problem
All dash ENI meter class stats were incorrectly written to COUNTERS_DB using the RID instead of the VID as the keys for the switch and ENI oid.

Fix
Use the Switch and ENI VID in the COUNTER_DB keys for dash meter class.

sonic-db-cli COUNTERS_DB hgetall 'COUNTERS:{"switch_id":"oid:0x21000000000000","eni_id":"oid:0x700800000001c","meter_class":"89"}'
{'SAI_METER_BUCKET_ENTRY_STAT_INBOUND_BYTES': '0', 'SAI_METER_BUCKET_ENTRY_STAT_OUTBOUND_BYTES': '210'}

**Problem**
All dash ENI meter class stats were incorrectly written to COUNTERS_DB using the RID instead of the VID as the keys for the switch and ENI oid.

**Fix**
Use the Switch and ENI VID in the COUNTER_DB keys for dash meter class.

```
sonic-db-cli COUNTERS_DB hgetall 'COUNTERS:{"switch_id":"oid:0x21000000000000","eni_id":"oid:0x700800000001c","meter_class":"89"}'
{'SAI_METER_BUCKET_ENTRY_STAT_INBOUND_BYTES': '0', 'SAI_METER_BUCKET_ENTRY_STAT_OUTBOUND_BYTES': '210'}
```
@mssonicbld
Copy link
Collaborator Author

Original PR: #1725

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny prsunny merged commit ec4a54d into sonic-net:202511 Feb 20, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants