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

Order ECMP HLD. #896

Merged
merged 6 commits into from
Dec 10, 2021
Merged

Order ECMP HLD. #896

merged 6 commits into from
Dec 10, 2021

Conversation

abdosi
Copy link
Contributor

@abdosi abdosi commented Nov 8, 2021

HLD for Order ECMP Feature. Nexthop in ECMP will be sorted/order and the order will be preserved when programming into ASIC via SAI.

PR that are merged for this Feature are:

  1. Enable/Disable Order ECMP feature. sonic-buildimage#9651
  2. OA changes to support Ordered ECMP and DVS test for same. sonic-swss#2092
  3. Added Support for enum query capability of Nexthop Group Type. sonic-sairedis#989

Signed-off-by: Abhishek Dosi abdosi@microsoft.com

abdosi and others added 4 commits November 8, 2021 22:55
Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
@abdosi abdosi requested a review from prsunny November 8, 2021 23:26
Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
@abdosi abdosi merged commit 78f3f73 into sonic-net:master Dec 10, 2021
@abdosi abdosi deleted the order_ecmp branch December 10, 2021 01:05
abdosi added a commit to sonic-net/sonic-buildimage that referenced this pull request Jan 7, 2022
Updated Jinja2 Template in switch.json.j2 for enabling/disabling Order ECMP feature based on device role.
Changes as per design: sonic-net/SONiC#896
abdosi added a commit to sonic-net/sonic-sairedis that referenced this pull request Jan 11, 2022
What/Why I did:
Added Support for enum query capability of Nexthop Group Type.
This is needed for DVS testing of Ordered ECMP Feature.
Design Doc: sonic-net/SONiC#896
@zhangyanzhao
Copy link
Collaborator

@abdosi would you please add the code PRs as what we asked?

@abdosi
Copy link
Contributor Author

abdosi commented Mar 25, 2022

@abdosi would you please add the code PRs as what we asked?

@zhangyanzhao Updated.

pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
…-net#989)

What/Why I did:
Added Support for enum query capability of Nexthop Group Type.
This is needed for DVS testing of Ordered ECMP Feature.
Design Doc: sonic-net/SONiC#896
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.

3 participants