[dualtor] Implement y_cable_simulator client#2652
Merged
bingwang-ms merged 3 commits intosonic-net:masterfrom Dec 15, 2020
Merged
[dualtor] Implement y_cable_simulator client#2652bingwang-ms merged 3 commits intosonic-net:masterfrom
bingwang-ms merged 3 commits intosonic-net:masterfrom
Conversation
This commit contains the code for y_cable_simulator client. The simulator driver is injected to pmon container in 'deploy_mg' for dualtor topo and hook the physical y_cable driver. Signed-off-by: bingwang <bingwang@microsoft.com>
wangxin
reviewed
Dec 11, 2020
bingwang-ms
force-pushed
the
y_cable_simulator_client
branch
from
8967b4e to
18a6c2d
Compare
1. Add necessary empty line 2. Update comments and argument names 3. Update exception handling logic. Signed-off-by: bingwang <bingwang@microsoft.com>
bingwang-ms
force-pushed
the
y_cable_simulator_client
branch
from
18a6c2d to
3cc5354
Compare
Collaborator
Author
|
@wangxin Thanks for your great suggestions. All updated. |
wangxin
reviewed
Dec 12, 2020
lguohan
reviewed
Dec 12, 2020
| @@ -0,0 +1,2 @@ | |||
| mux_simulator_port: 8080 | |||
Contributor
There was a problem hiding this comment.
maybe we have mux_simulator per topology?
Signed-off-by: bingwang <bingwang@microsoft.com>
Collaborator
Author
|
Thanks @wangxin . All updated. |
wangxin
approved these changes
Dec 14, 2020
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
``` 39cdb49c7 [202012][show] Add bgpraw to show run all (sonic-net#2639) b3ebba2ca [202012][show] add new CLI to show tunnel route objects sonic-net#2255 (sonic-net#2659) d08f59b9f Fixed a bug in "show vnet routes all" causing screen overrun. (sonic-net#2644) (sonic-net#2654) a996abdb5 [202012][show] show logging CLI support for logs stored in tmpfs (sonic-net#2652) c60f771c0 [202012][show_bfd] add local discriminator in show bfd command (sonic-net#2616) ```
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: bingwang bingwang@microsoft.com
Description of PR
Summary:
Fixes # (issue)
This PR is to introduce a simulator for y_cable for testing.
The simulator driver is injected into
pmoncontainer indeploy_mgfordualtortopo and hook the physical y_cable driver. The simulator interact with simulator driver running on vmhost server, and control the behavior of y_cable with open-flow.Type of change
Approach
What is the motivation for this PR?
This commit contains the code for y_cable_simulator client.
How did you do it?
The simulator driver is injected to
pmoncontainer in 'deploy_mg' for dualtor topo and hook the physical y_cable driver. The simulator interact with simulator driver running on vmhost server, and control the behavior of y_cable with open-flow.How did you verify/test it?
Verified on a dual-tor topo.
Any platform specific information?
No.
Supported testbed topology if it's a new test case?
Only applicable to dualtor topo.
Documentation