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

Update db_migrator to support pfcwd_sw_enable #2087

Merged
merged 3 commits into from
Apr 5, 2022

Conversation

bingwang-ms
Copy link
Contributor

@bingwang-ms bingwang-ms commented Mar 1, 2022

Signed-off-by: bingwang bingwang@microsoft.com

What I did

This PR is to update db_migrator to support pfcwd_sw_enable.
Currently, table pfc_enable is to specify on which queue to enable PFC and pfc watchdog. As we are going to add two extra lossless queues on which on watchdog is enable(HLD sonic-net/SONiC#950), a new table is required to specify on which queue to enable PFC watchdog. That is pfcwd_sw_enable.

Table Description
pfc_enable Specify on which queues to enable PFC
pfcwd_sw_enable Specify on which queues to enable software PFC watchdog

HLD DSCP remapping

Change in orchagent Update orchagent to support new field pfcwd_sw_enable

How I did it

Update db_migrator.

How to verify it

  1. Verified by UT
  2. Verified by copying the updated db_migrator.py to a SONiC box and run db_migrator.py -o migrate

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)

Signed-off-by: bingwang <bingwang@microsoft.com>
@bingwang-ms
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms bingwang-ms marked this pull request as ready for review March 7, 2022 11:18
@bingwang-ms
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@neethajohn
Copy link
Contributor

@bingwang-ms , please update the description to not refer to the 'pfcwd_hw_enable' field

@neethajohn neethajohn merged commit 9e2fbf4 into sonic-net:master Apr 5, 2022
@qiluo-msft
Copy link
Contributor

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

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