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

[201803][reboot] log reboot progress and add sanity check #527

Open
wants to merge 1 commit into
base: 201803
Choose a base branch
from

Conversation

yxieca
Copy link
Contributor

@yxieca yxieca commented May 8, 2019

- What I did

  • Do not reboot device if the next image is missing.
  • Log if system is read-only or full.
  • Log user reboot requests since we don't have reboot-cause in 301803.

Signed-off-by: Ying Xie ying.xie@microsoft.com

- How to verify it
Verified that reboot process is logged in syslog.
Verified that read-only file system will generate log but don't prevent reboot.
verified that non-existing next-boot image will stop reboot.

- Do not reboot device if the next image is missing.
- Log if system is read-only or full.
- Log user reboot requests since we don't have reboot-cause in 301803.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@yxieca yxieca requested a review from lguohan May 8, 2019 02:38
@jleveque
Copy link
Contributor

Retest this please

3 similar comments
@qiluo-msft
Copy link
Contributor

Retest this please

@jleveque
Copy link
Contributor

Retest this please

@jleveque
Copy link
Contributor

Retest this please

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.

3 participants