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

[Reclaiming buffer][202106] Database migrator for reclaiming buffer #1897

Merged
merged 7 commits into from
Dec 20, 2021

Conversation

stephenxs
Copy link
Collaborator

This is to cherry-pick community PR #1822 to 202106.

What I did

Db migrator support reclaiming reserved buffer for unused ports

As there is no empty slot for database version between 202106 and master (202111), this migration will be done regardless of whether database version is changed. The DB migrator should be idempotent.

Signed-off-by: Stephen Sun [email protected]

How I did it

For admin down ports, if the buffer objects configuration aligns with default configuration, set the buffer objects configuration as:

  • Dynamic model: all normal buffer objects are configured on admin down ports. Buffer manager will apply zero profiles on admin down ports.
  • Static model: zero buffer objects are configured on admin down ports.

How to verify it

  1. Unit test.
  2. Manually test.

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@stephenxs
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Collaborator Author

Also cherry-picked a missing logic to test whether it's a double pool module or single pool module.

@stephenxs
Copy link
Collaborator Author

/azpw run Azure.sonic-utilities

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-utilities

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik merged commit 642f508 into sonic-net:202106 Dec 20, 2021
@stephenxs stephenxs deleted the reclaim-buffer-202106 branch December 20, 2021 12:45
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