Skip to content
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

[action] [PR:19358] Support yang model for buffer pool percentage (#19358) #19740

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

mssonicbld
Copy link
Collaborator

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 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
@mssonicbld
Copy link
Collaborator Author

Original PR: #19358

@dprital
Copy link
Collaborator

dprital commented Aug 22, 2024

@bingwang-ms , Can you please why this cherry picked was stuck ?

@bingwang-ms
Copy link
Contributor

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Contributor

I just retriggered the PR test.

@bingwang-ms bingwang-ms enabled auto-merge (squash) September 6, 2024 21:31
@bingwang-ms
Copy link
Contributor

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld closed this Sep 6, 2024
auto-merge was automatically disabled September 6, 2024 23:01

Pull request was closed

@bingwang-ms bingwang-ms reopened this Sep 11, 2024
@bingwang-ms bingwang-ms enabled auto-merge (squash) September 11, 2024 00:32
@mssonicbld mssonicbld merged commit 3c52766 into sonic-net:202405 Sep 11, 2024
23 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.

4 participants