Warm reboot: Extract the addToSync() function from Consumer::execute(), and new Or…#546
Closed
jipanyang wants to merge 3 commits intosonic-net:masterfrom
Closed
Warm reboot: Extract the addToSync() function from Consumer::execute(), and new Or…#546jipanyang wants to merge 3 commits intosonic-net:masterfrom
jipanyang wants to merge 3 commits intosonic-net:masterfrom
Conversation
…ch::addExistingData() Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
prsunny
reviewed
Jul 27, 2018
orchagent/orch.cpp.orig
Outdated
| @@ -0,0 +1,495 @@ | |||
| #include <fstream> | |||
Collaborator
There was a problem hiding this comment.
Why this file? orch.cpp.orig?
Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
This was referenced Jul 27, 2018
jipanyang
changed the title
Contributor
Author
EdenGri
pushed a commit
to EdenGri/sonic-swss
that referenced
this pull request
Feb 28, 2022
…onic-net#546) * [config/mlnx.py] Prompt the risk of excessive consumption of disk space when user intends to turn on the sdk sniffer. The SDK sniffer is a diagnosis feature, introduced with a view to recording the all the operations to SDK into the sniffer file which the low level team can replay and investigate. Only the sniffer file contains all the operations from SWSS restart does it help. To handle it in a way like log-rotate can hurt its integrity since the operations can be lost during rotating and to address this requires big effort. So we just add a promot of that risk for now. * [config/mlnx.py] adjust prompt. adjust prompt in order to make it more clear. * [config/mlnx.py] adjust prompt. adjust prompt in order to make it more clear. (recommit this in order to trigger vstest of the sonic-utilities and check whether it can pass)
oleksandrivantsiv
pushed a commit
to oleksandrivantsiv/sonic-swss
that referenced
this pull request
Mar 1, 2023
… run vs pytests) (sonic-net#546) Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
jianyuewu
pushed a commit
to jianyuewu/sonic-swss
that referenced
this pull request
Dec 24, 2025
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
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.
…ch::addExistingData()
Signed-off-by: Jipan Yang jipan.yang@alibaba-inc.com
What I did
Add support to explicitly read existing data from a table and add them to the m_toSync of consumer in this Orch
Why I did it
Needed for state restore of orchagent during warm restart.
How I verified it
VS testing.
Details if related