Skip to content

resource monitoring support for generic SAI extension tables#2649

Merged
svshah-intel merged 2 commits intosonic-net:masterfrom
svshah-intel:master
Jun 7, 2023
Merged

resource monitoring support for generic SAI extension tables#2649
svshah-intel merged 2 commits intosonic-net:masterfrom
svshah-intel:master

Conversation

@svshah-intel
Copy link
Copy Markdown
Contributor

High Level Design: sonic-net/SONiC#1243

The change incudes,

  • new resource type to reprsent all extension tables
  • countermap for each extension table under this new resource type
  • extension table mgr to call APIs to track usage counter of entries in each extension table
  • pytest automation

Signed-off-by: svshah-intel shitanshu.shah@intel.com

@reshmaintel
Copy link
Copy Markdown

@prsunny @volodymyrsamotiy Could you please review/approve this PR for 202305. Thanks.

@reshmaintel
Copy link
Copy Markdown

reshmaintel commented May 18, 2023

@oleksandrivantsiv @prsunny Also please review this PR. Thank you.
@zhangyanzhao

@prsunny prsunny requested a review from oleksandrivantsiv June 1, 2023 15:47
Copy link
Copy Markdown
Collaborator

@prsunny prsunny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@prsunny
Copy link
Copy Markdown
Collaborator

prsunny commented Jun 6, 2023

@oleksandrivantsiv , @volodymyrsamotiy , please sign off

@svshah-intel svshah-intel merged commit 2969488 into sonic-net:master Jun 7, 2023
theasianpianist pushed a commit to theasianpianist/sonic-swss that referenced this pull request Jul 20, 2023
…et#2649)

* resource monitoring support for generic SAI extension tables

* fix p4orch_test build
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
…et#2649)

* resource monitoring support for generic SAI extension tables

* fix p4orch_test build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants