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.

Sign up for GitHub

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

Jump to bottom

[doc] active-active dual ToR HLD #1005

Merged
merged 15 commits into from
Jul 14, 2022
Merged

[doc] active-active dual ToR HLD #1005

merged 15 commits into from
Jul 14, 2022

Conversation

zjswhhh
Copy link
Contributor

@zjswhhh zjswhhh commented May 23, 2022 •
edited
Loading

Submitting active-active dual ToR HLD initial revision.

Repo PR title State
sonic-linkmgrd Add ActiveActiveStateMachine implementation GitHub issue/pull request detail
sonic-platform-daemons grpc client implementation for active-active dualtor GitHub issue/pull request detail
sonic-mgmt [dualtor] Add nic_simulator GitHub issue/pull request detail
sonic-swss-common Add docker-mux related table names GitHub issue/pull request detail

Sign-off: Jing Zhang zhangjing@microsoft.com

@zjswhhh zjswhhh changed the title [doc] active-actived dual ToR HLD [doc] active-active dual ToR HLD May 23, 2022
@zjswhhh zjswhhh force-pushed the dualToR branch 2 times, most recently from 8dc505b to a115fc8 Compare May 27, 2022 18:44
@zjswhhh zjswhhh requested a review from prsunny May 27, 2022 18:44
@zjswhhh
updates: add IPinIP tunnel; add DB changes; add active-standby compar…
6101390 
…ison; re-add default route in decision table; add content table; add flow diagrams.
lolyu
lolyu previously approved these changes May 29, 2022
View reviewed changes
@zhangyanzhao
Copy link
Collaborator

@zjswhhh would you please add the code PRs by referring to EVPN VxLAN update for platforms using P2MP tunnel based L2 forwarding by dgsudharsan · Pull Request #806 · Azure/SONiC (github.com) as an example

@zjswhhh
Copy link
Contributor Author

zjswhhh commented Jun 8, 2022

@zjswhhh would you please add the code PRs by referring to EVPN VxLAN update for platforms using P2MP tunnel based L2 forwarding by dgsudharsan · Pull Request #806 · Azure/SONiC (github.com) as an example

Added.

lolyu
lolyu previously approved these changes Jun 10, 2022
View reviewed changes
lolyu
lolyu approved these changes Jul 13, 2022
View reviewed changes
Copy link
Contributor

@lolyu lolyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zjswhhh zjswhhh merged commit 42024fa into sonic-net:master Jul 14, 2022
@zjswhhh zjswhhh deleted the dualToR branch July 14, 2022 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rlhui rlhui rlhui left review comments

@kperumalbfn kperumalbfn kperumalbfn left review comments

@lguohan lguohan lguohan left review comments

@lolyu lolyu lolyu approved these changes

@yxieca yxieca yxieca approved these changes

@zhangyanzhao zhangyanzhao Awaiting requested review from zhangyanzhao

@prsunny prsunny Awaiting requested review from prsunny

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@zjswhhh @zhangyanzhao @lguohan @yxieca @kperumalbfn @lolyu @rlhui