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

[202405][wcmp]: Add W-ECMP CLI #3407

Open
wants to merge 1 commit into
base: 202405
Choose a base branch
from

Conversation

nazariig
Copy link
Collaborator

@nazariig nazariig commented Jul 9, 2024

Signed-off-by: Nazarii Hnydyn nazariig@nvidia.com

HLD: sonic-net/SONiC#1629

What I did

  • Implemented CLI for Weighted-Cost Multi-Path feature

How I did it

  • Integrated Weighted-Cost Multi-Path interface into config and show CLI root

How to verify it

  1. Run Weighted-Cost Multi-Path CLI UTs

Details if related

Previous command output (if the output of a command-line utility has changed)

  • N/A

New command output (if the output of a command-line utility has changed)

admin@sonic:/home/admin# show bgp device-global
TSA      WCMP
-------  -------
enabled  enabled
admin@sonic:/home/admin# show bgp device-global --json
{
    "tsa": "enabled",
    "wcmp": "enabled"
}

Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@nazariig nazariig changed the title [wcmp]: Add W-ECMP CLI [202405][wcmp]: Add W-ECMP CLI Jul 9, 2024
@nazariig nazariig marked this pull request as ready for review July 10, 2024 08:27
@nazariig
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@volodymyrsamotiy
Copy link
Collaborator

@bingwang-ms, FYI (separate 202405 PRs for cherry-picking it)

@bingwang-ms
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nazariig
Copy link
Collaborator Author

@bingwang-ms just a kind reminder

@liat-grozovik
Copy link
Collaborator

@bingwang-ms can you please help to merge?

@nazariig
Copy link
Collaborator Author

nazariig commented Oct 8, 2024

@liat-grozovik can you please help to merge?

@nazariig nazariig requested a review from moshemos October 11, 2024 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants