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

[cbf] Add Class Based Forwarding support #909

Merged
merged 2 commits into from
Oct 5, 2021

Conversation

abanu-ms
Copy link
Contributor

  • Added support for MAP_LIST serialization and deserialization

  • Add FC to QoS maps

  • Added a UT to validate the changes

  • Add NHG map API implementation

  • Add VS object availability for NHG maps

Signed-off-by: Alexandru Banu v-albanu@microsoft.com

@abanu-ms
Copy link
Contributor Author

This PR relies on Cisco's opencomputeproject/SAI#1193 PR. I'm not sure which is the proper way to update the SAI submodule so the PR pipeline works. Should I include it (once it's in SAI/master) in the code changes as well?

kcudnik
kcudnik previously approved these changes Sep 3, 2021
@kcudnik
Copy link
Collaborator

kcudnik commented Sep 3, 2021

please fix build process and resolve conflicts

@abanu-ms
Copy link
Contributor Author

abanu-ms commented Sep 6, 2021

In order to fix the build we need a newer version of SAI. We're going to wait for the official SAI module update before merging this in.

* Added support for MAP_LIST serialization and deserialization

* Added a UT to validate the changes

* Add NHG map API implementation

* Add VS object availability for NHG maps

Signed-off-by: Alexandru Banu <v-albanu@microsoft.com>
@abanu-ms
Copy link
Contributor Author

abanu-ms commented Oct 5, 2021

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@TACappleman
Copy link
Contributor

@kcudnik could you review this change, now that the SAI dependency issue is solved?

@TACappleman
Copy link
Contributor

@kcudnik Thanks - can this also be merged? (We don't have permissions!)

@kcudnik kcudnik merged commit ecb95a8 into sonic-net:master Oct 5, 2021
@abanu-ms abanu-ms deleted the cbf_pr branch October 6, 2021 08:15
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
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