Skip to content

Comments

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

Merged
mssonicbld merged 1 commit intoAzure:202506from
mssonicbld:cherry/msft-202506/1725
Jan 22, 2026
Merged

[action] [PR:1725] Fix dash meter COUNTERS_DB keys to use VID instead of RID#103
mssonicbld merged 1 commit intoAzure:202506from
mssonicbld:cherry/msft-202506/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: sonic-net/sonic-sairedis#1725

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@mssonicbld mssonicbld merged commit 7a5e16b into Azure:202506 Jan 22, 2026
5 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.

1 participant