Skip to content

Order ECMP HLD.#896

Merged
abdosi merged 6 commits intosonic-net:masterfrom
abdosi:order_ecmp
Dec 10, 2021
Merged

Order ECMP HLD.#896
abdosi merged 6 commits intosonic-net:masterfrom
abdosi:order_ecmp

Conversation

@abdosi
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

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

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