Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[warm-boot] Fix the issue as reported in https://github.com/Azure/sonic-buildimage/issues/5255 #5315

Merged
merged 1 commit into from
Sep 4, 2020

Conversation

abdosi
Copy link
Contributor

@abdosi abdosi commented Sep 4, 2020

Why I did:

Fixes the issue #5255

Root Cause: Waiting on Restore count != 0 can lead to race condition
between orchagent process and swssconfig.sh.

Ideally check of Restore count != 0 is not needed as the State DB
cannot be flushed as if it was flushed then Warm Restart or swss-restart
enable flag should not be true also.

How I verify:
After removing check did not hit race condition that resulted the error.

sonic-net#5255

Root Cause: Waiting on Restore count != 0 can lead to race condition
between orchagent process and swssconfig.sh.

Ideally check of  Restore count != 0 is not needed as the State DB
cannot be flushed as if it was flushed then Warm Restart or swss-restart
should not be true also.
@abdosi abdosi merged commit eff745f into sonic-net:master Sep 4, 2020
@abdosi abdosi deleted the warmboot_fixes branch September 4, 2020 16:34
abdosi added a commit that referenced this pull request Sep 6, 2020
#5255

Root Cause: Waiting on Restore count != 0 can lead to race condition
between orchagent process and swssconfig.sh.

Ideally check of  Restore count != 0 is not needed as the State DB
cannot be flushed as if it was flushed then Warm Restart or swss-restart
should not be true also.
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
sonic-net#5255

Root Cause: Waiting on Restore count != 0 can lead to race condition
between orchagent process and swssconfig.sh.

Ideally check of  Restore count != 0 is not needed as the State DB
cannot be flushed as if it was flushed then Warm Restart or swss-restart
should not be true also.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[201911] Warmboot fails on Arista 7050
2 participants