[doc]: Add gnmi feedback design for SmartSwitch #1759
Conversation
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
Pterosaur
requested review from
lguohan,
qiluo-msft,
prsunny,
prabhataravind and
ganglyu
Signed-off-by: Ze Gan <[email protected]>
ganglyu
reviewed
Jul 30, 2024
|
|
||
| ### APPL STATE DB | ||
|
|
||
| DASH state table in APPL STATE DB, table name is the same as DASH table, key is the same, and fields are result and version ID if it's not a child object. |
There was a problem hiding this comment.
Can we add an example for DASH_ROUTE_GROUP_TABLE?
There was a problem hiding this comment.
Here is just an example, more tables will be on the dash hld document.
Signed-off-by: Ze Gan <[email protected]>
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
Signed-off-by: Ze Gan <[email protected]>
Pterosaur
force-pushed
the
gnmi_feedback
branch
from
d3ba5fd to
59fd5cc
Compare
|
/azp run |
|
No pipelines are associated with this pull request. |
Signed-off-by: Ze Gan <[email protected]>
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
|
@prabhataravind , could you link your implementation PR into description? |
|
Hi @qiluo-msft , could you please help to sign off this PR? This has been reviewed by community as well. |
prsunny
pushed a commit
to sonic-net/sonic-swss
that referenced
this pull request
Apr 28, 2025
…3490) * [orchagent/dash]: Changes to support gNMI feedback for DASH objects This PR makes the following changes: Write result of SAI API call to APP_STATE_DB for DASH objects as per sonic-net/SONiC#1759 Instantiate DpuOrchDaemon for smartswitch DPUs. Related PRs [doc]: Add gnmi feedback design for SmartSwitch SONiC#1759 [doc]: Init Smartswitch database High Level Design SONiC#1534 [dbconnect]: Support DPU database schema sonic-swss-common#845 What I did Changes to support gNMI feedback path for DASH objects on SmartSwitch DPUs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This is the initial version of the GNMI feedback design for SmartSwitch. It includes the sequence flows between GNMI client and server and between SONiC internal components on NPU and DPU.
swss changes: sonic-net/sonic-swss#3490