sonic-utilities
sonic-utilities copied to clipboard
sonic-net
Add SAG implementation
What I did
- Add static anycast gateway related commands support. Refer to SAG HLD#837
How I did it
- Add new commands
- config static-anycast-gateway mac_address add/del
- config vlan static-anycast-gateway add/del
- Add unit test cases
How to verify it
- Executing unit tests
Signed-off-by: Jimi Chen [email protected]
@cbpaviz Thanks for your comments, codes are modified according that. Please help to check and review again.
![azure-pipelines[bot] avatar](https://avatars.githubusercontent.com/in/9426?v=4 "Published by a pub.dev verified publisher"){kind=small}
Commenter does not have sufficient privileges for PR 1887 in repo Azure/sonic-utilities
@cbpaviz Could you help to review again, I merged the code from master branch due to some confict.
@cbpaviz Could you help to review again, I merged the code from master branch due to some conflict.
@superchild , why SAG config commands are different compare to EC config commands (available in their config guid). is it intentional ?
@kulpatel, this PR is discussed and reviewed by community, EC will also follow the same design after this merged. The original design is slightly different from community's opinion.
@zhangyanzhao Sorry for the wrong operation sync, this PR's commit history is gone. I already reset the commit on my dev branch, could you please reopen this PR? Or should I resend a new PR for it?