Skip to content

[cbf] Add Class Based Forwarding support - #909

Merged
kcudnik merged 2 commits into
sonic-net:masterfrom
Metaswitch:cbf_pr
Oct 5, 2021
Merged

kcudnik merged 2 commits into
sonic-net:masterfrom
Metaswitch:cbf_pr

Conversation

@abanu-ms

Copy link
Copy Markdown
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
Copy Markdown
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

kcudnik commented Sep 3, 2021

Copy link
Copy Markdown
Collaborator

please fix build process and resolve conflicts

@abanu-ms

abanu-ms commented Sep 6, 2021

Copy link
Copy Markdown
Contributor Author

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

abanu-ms commented Oct 5, 2021

Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld

Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@TACappleman

Copy link
Copy Markdown
Contributor

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

@TACappleman

Copy link
Copy Markdown
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