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 reboot] Skip ASIC config pre-check if current image does not support it #637

Merged
merged 1 commit into from
Sep 9, 2019

Conversation

daall
Copy link
Contributor

@daall daall commented Sep 9, 2019

What I did
I edited the ASIC config check script to abort the check if the current image does not have a checksum.

How to verify it

  1. Run warm-reboot with the updated check script on a SONiC device that does not contain an asic_config_checksum file.
  2. Confirm that warm-reboot proceeds past the pre-check stage.
  3. Confirm that the syslog contains messages indicating that there was no checksum found on the current image and that the pre-check is not supported.

Signed-off-by: Danny Allen daall@microsoft.com

…pport it

Signed-off-by: Danny Allen <daall@microsoft.com>
@daall daall requested review from lguohan and yxieca September 9, 2019 19:20
@yxieca yxieca merged commit a6a44e8 into sonic-net:master Sep 9, 2019
yxieca pushed a commit that referenced this pull request Sep 9, 2019
…pport it (#637)

Signed-off-by: Danny Allen <daall@microsoft.com>
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
Update the meta code to support DNAT Pool changes (sonic-net#616)
[syncd] Fix notification on shutdown request (sonic-net#637)
Advance the submodule head of SAI (sonic-net#641)

Signed-off-by: Stephen Sun <stephens@mellanox.com>
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
Update the meta code to support DNAT Pool changes (sonic-net#616)
[syncd] Fix notification on shutdown request (sonic-net#637)
Advance the submodule head of SAI (sonic-net#641)
Add new line in sai_meta_log_syncd fprintf call (sonic-net#649)
Fix Warmboot Issue when upgraded Image SAI return Switch Internal
OID not accounted in previous image. (sonic-net#654)
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.

2 participants