Skip to content

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

Merged
mssonicbld merged 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/19358
Sep 11, 2024
Merged

[action] [PR:19358] Support yang model for buffer pool percentage (#19358)#19740
mssonicbld merged 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/19358

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

@azure-pipelines
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

@azure-pipelines
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
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