grpc client implementation for active-active dualtor#248
Merged
vdahiya12 merged 45 commits intosonic-net:masterfrom May 31, 2022
Merged
grpc client implementation for active-active dualtor#248vdahiya12 merged 45 commits intosonic-net:masterfrom
vdahiya12 merged 45 commits intosonic-net:masterfrom
Conversation
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
…ting the path to install Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
|
This pull request introduces 8 alerts when merging 81753fe into f09bd31 - view on LGTM.com new alerts:
|
vdahiya12
changed the title
vdahiya12
pushed a commit
to vdahiya12/sonic-platform-daemons
that referenced
this pull request
Apr 4, 2022
* SFP-Refactor: bug fix
Signed-off-by: vaibhav-dahiya <[email protected]>
|
This pull request introduces 3 alerts when merging ffcaf35 into 9ac12bf - view on LGTM.com new alerts:
|
Signed-off-by: vaibhav-dahiya <[email protected]>
|
This pull request introduces 38 alerts when merging 3a496fb into e0f8a35 - view on LGTM.com new alerts:
|
Signed-off-by: vaibhav-dahiya <[email protected]>
|
This pull request introduces 3 alerts when merging 67d9408 into e0f8a35 - view on LGTM.com new alerts:
|
Signed-off-by: vaibhav-dahiya <[email protected]>
|
This pull request introduces 1 alert when merging 771ca40 into e0f8a35 - view on LGTM.com new alerts:
|
Signed-off-by: vaibhav-dahiya <[email protected]>
|
This pull request introduces 1 alert when merging 2d82162 into ce217c0 - view on LGTM.com new alerts:
|
Signed-off-by: vaibhav-dahiya <[email protected]>
|
This pull request introduces 1 alert when merging 812a768 into ce217c0 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging 8d1007f into ce217c0 - view on LGTM.com new alerts:
|
Signed-off-by: vaibhav-dahiya <[email protected]>
|
This pull request introduces 2 alerts when merging 51862c4 into ce217c0 - view on LGTM.com new alerts:
|
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
lolyu
previously approved these changes
May 24, 2022
into test-grpc Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
lolyu
previously approved these changes
May 25, 2022
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
lolyu
previously approved these changes
May 27, 2022
into test-grpc Signed-off-by: vaibhav-dahiya <[email protected]>
Signed-off-by: vaibhav-dahiya <[email protected]>
lolyu
approved these changes
May 28, 2022
vdahiya12
added a commit
that referenced
this pull request
Jun 1, 2022
This PR is to support the gRPC interaction with to the SoC/Nic-Simulator for link manager state machine ActiveActiveStateMachine to work if the port is configured as active-active cable type. It supports the RPC's support for SONiC by creating channels/stubs when a cable is pulled in as well as when the ycabled is initialized from supervisord. The logic to treat a cable/port as "active-active" comes from minigraph/config_db and then this PR has the logic to take care of serving RPC's as requested by other daemons. It does the RPC call when an appropriate request lands the ycabled as described below The following Tables are served by ycabled for gRPC RPC by listening to changes in app DB request from linkmgr/orchagent and corresponding results are written to state DB. HW_MUX_CABLE_TABLE HW_MUX_TABLE_TABLE_PEER This PR also adds logic to listening to the forwarding state command table and get the response back from gRPC and write to forwarding state response FORWARDING_STATE_COMMAND -> FORWARDING_STATE_RESPONSE for getting the forwarding state request/response using gRPC This PR also has logic for gRPC library build using build_ext extension. The proto definition is present in proto/proto_out/ The setup.py changes make sure that gRPC libs are generated correctly. the corresponding gRPC libs are generated in proto_out directory in python packages directory and are imported by ycabled Motivation and Context DualToR active-active support for gRPC interface to support the state machine How Has This Been Tested? Unit-Tests and deploying changes on a DualToR testbed
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
2 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.
Description
This PR is to support the gRPC interaction with to the SoC/Nic-Simulator for link manager state machine ActiveActiveStateMachine to work if the port is configured as active-active cable type.
It supports the RPC's support for SONiC by creating channels/stubs when a cable is pulled in as well as when the ycabled is initialized from supervisord. The logic to treat a cable/port as "active-active" comes from minigraph/config_db and then this PR has the logic to take care of serving RPC's as requested by other daemons.
It does the RPC call when an appropriate request lands the ycabled as described below
The following Tables are served by ycabled for gRPC RPC by listening to changes in app DB request from linkmgr/orchagent and corresponding results are written to state DB.
HW_MUX_CABLE_TABLEHW_MUX_TABLE_TABLE_PEERThis PR also adds logic to listening to the forwarding state command table and get the response back from gRPC and write to forwarding state response
FORWARDING_STATE_COMMAND->FORWARDING_STATE_RESPONSEfor getting the forwarding state request/response using gRPC
This PR also has logic for gRPC library build using build_ext extension. The proto definition is present in
proto/proto_out/The setup.py changes make sure that gRPC libs are generated correctly.
the corresponding gRPC libs are generated in proto_out directory in python packages directory and are imported by ycabled
Motivation and Context
DualToR active-active support for gRPC interface to support the state machine
How Has This Been Tested?
Unit-Tests and deploying changes on a DualToR testbed
Additional Information (Optional)