-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support yang model for buffer pool percentage #19358
Support yang model for buffer pool percentage #19358
Conversation
/azpw run azure.sonic-buildimage |
/AzurePipelines run azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
/azpw ms_conflict |
/azpw run azure.sonic-buildimage |
/AzurePipelines run azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
fed09c0
to
6e6617e
Compare
Failure
|
/azpw run azure.sonic-buildimage |
/AzurePipelines run azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
/azpw run azure.sonic-buildimage |
/AzurePipelines run azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
/azpw run azure.sonic-buildimage |
/AzurePipelines run azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
6e6617e
to
2e3ae0a
Compare
/azpw run azure.sonic-builldimage |
/AzurePipelines run azure.sonic-builldimage |
/azpw run azure.sonic-builldimage |
/AzurePipelines run azure.sonic-builldimage |
src/sonic-yang-models/tests/yang_model_tests/tests_config/buffer_pool.json
Show resolved
Hide resolved
Do you have any doc for this dynamic buffer mode? |
|
Signed-off-by: Stephen Sun <stephens@nvidia.com>
src/sonic-yang-models/tests/yang_model_tests/tests_config/buffer_pool.json
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@bingwang-ms , Can you please cherry pick to 202405 ? |
#### Why I did it Support yang model for `percentage` field in `BUFFER_POOL` table. It is used in dynamic buffer model only and represents the percentage of a buffer pool's size compared to the available memory size #### How I did it #### How to verify it Unit test and manual test
Cherry-pick PR to 202405: #19740 |
#### Why I did it Support yang model for `percentage` field in `BUFFER_POOL` table. It is used in dynamic buffer model only and represents the percentage of a buffer pool's size compared to the available memory size #### How I did it #### How to verify it Unit test and manual test
@stephenxs cherry pick PR didn't pass PR checker. Please check!!! Auto cherry pick PR will be closed in 3 days. |
#### Why I did it Support yang model for `percentage` field in `BUFFER_POOL` table. It is used in dynamic buffer model only and represents the percentage of a buffer pool's size compared to the available memory size #### How I did it #### How to verify it Unit test and manual test
Why I did it
Support yang model for
percentage
field inBUFFER_POOL
table.It is used in dynamic buffer model only and represents the percentage of a buffer pool's size compared to the available memory size
Work item tracking
How I did it
How to verify it
Unit test and manual test
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)