Skip to content

dual tor topology#2333

Merged
lguohan merged 2 commits intosonic-net:masterfrom
lguohan:dualtor
Oct 20, 2020
Merged

dual tor topology#2333
lguohan merged 2 commits intosonic-net:masterfrom
lguohan:dualtor

Conversation

@lguohan
Copy link
Contributor

@lguohan lguohan commented Oct 13, 2020

Signed-off-by: Guohan Lu [email protected]

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

add support for dual-tor topology

  • introduce dualtor topology
  • introduce simulated muxy cable via ovs switch
  • support deploy-mg for multi-dut
  • support used-defined loopback ip address via topology file

How did you do it?

details in the pr

How did you verify/test it?

tested following commands on both single-dut and multi-dut topology (all on kvm)

  • add-topo/remove-top/disconnect-vms/connect-vms/refresh-dut

Any platform specific information?

Supported testbed topology if it's a new test case?

no new test cases introduce

Documentation

@lguohan lguohan requested a review from lolyu October 13, 2020 01:32
@lgtm-com
Copy link

lgtm-com bot commented Oct 13, 2020

This pull request introduces 1 alert when merging 055dce9a380d88de3190b00410c883836631ca39 into 552dac9 - view on LGTM.com

new alerts:

  • 1 for Variable defined multiple times

@lguohan
Copy link
Contributor Author

lguohan commented Oct 15, 2020

retest this please

@lguohan lguohan force-pushed the dualtor branch 2 times, most recently from 71191ff to ee25c16 Compare October 16, 2020 12:28
@lgtm-com
Copy link

lgtm-com bot commented Oct 17, 2020

This pull request fixes 1 alert when merging 45d90a528ddd72ad79e90bd6134c28694de9f5cb into bdfd25b - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 18, 2020

This pull request fixes 1 alert when merging 9a662456ee478596e5dce7262d1a182d53660f2e into bdfd25b - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 18, 2020

This pull request fixes 1 alert when merging 6ea3f1d77b388a0f765be7b685bff69ffaae743a into bdfd25b - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lguohan lguohan marked this pull request as ready for review October 19, 2020 04:02
@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2020

This pull request fixes 1 alert when merging 6324366b0ba63d89f41a016bea24ebfb3dd956ea into bdfd25b - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2020

This pull request fixes 1 alert when merging aa7f6b61c90aa231b7378d6db9fae286700f4ad5 into bdfd25b - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2020

This pull request fixes 1 alert when merging 00631a038f7faa4052c6c9ac58ca67a6bd368cfe into d67a148 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2020

This pull request fixes 1 alert when merging 55526bb37e9cd4cb80c5b6e765f1c204b50ae4b3 into 3b152b4 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

- introduce dualtor topology
- introduce simulated muxy cable via ovs switch
- support deploy-mg for multi-dut

Signed-off-by: Guohan Lu <[email protected]>
@lgtm-com
Copy link

lgtm-com bot commented Oct 20, 2020

This pull request fixes 1 alert when merging 89f8990 into 3b152b4 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lguohan
Copy link
Contributor Author

lguohan commented Oct 20, 2020

test error not related to this pr.

@lguohan lguohan merged commit de60fbf into sonic-net:master Oct 20, 2020
@lguohan lguohan deleted the dualtor branch October 20, 2020 05:52
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Update sonic-utilities submodule pointer to include the following:
* [route_check]: Ignore standalone tunnel routes (sonic-net#2325) ([sonic-net#2346](sonic-net/sonic-utilities#2346))
* [VRF]Adding CLI checks to ensure Vrf is valid in interface bind and static route commands ([sonic-net#2333](sonic-net/sonic-utilities#2333))
* Subinterface vrf bind issue fix ([sonic-net#2211](sonic-net/sonic-utilities#2211))
* [decode-syseeprom] Fix setting use_db based on support_eeprom_db ([sonic-net#2270](sonic-net/sonic-utilities#2270))
* Fix vrf UT failed issue ([sonic-net#2309](sonic-net/sonic-utilities#2309))

Signed-off-by: dprital <[email protected]>

Signed-off-by: dprital <[email protected]>
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.

4 participants