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

firmware: upgrade_needs_reboot flag is overloaded #5358

Closed
2 tasks done
fichtner opened this issue Nov 16, 2021 · 1 comment
Closed
2 tasks done

firmware: upgrade_needs_reboot flag is overloaded #5358

fichtner opened this issue Nov 16, 2021 · 1 comment
Assignees
Labels
cleanup Low impact changes
Milestone

Comments

@fichtner
Copy link
Member

Important notices

Before you add a new report, we ask you kindly to acknowledge the following:

Describe the bug

When upgrade_needs_reboot is propagated to the GUI nowadays a major update will force the flag as well. But when there are minor updates pending that don't require a reboot the system will perform the minor update (as it should) but claim it requires a reboot which it does not.

To Reproduce

Visible on development track with sticky beta upgrade possibility.

Expected behavior

Separate reboot flags. The larger issue is that upgrade_needs_reboot is actually for updates only but used in code already so renaming it and splitting the flags in two is difficult.

Describe alternatives you considered

Back out 19a927c and let it be implied that set upgrades always require a reboot.

Screenshots

N/A

Relevant log files

N/A

Additional context

N/A

Environment

22.1.b

@fichtner fichtner added the cleanup Low impact changes label Nov 16, 2021
@fichtner fichtner added this to the 22.1 milestone Nov 16, 2021
@fichtner fichtner self-assigned this Nov 16, 2021
fichtner added a commit that referenced this issue Nov 17, 2021
Keep old flag in place to not break any API consumers for a while.
fichtner added a commit that referenced this issue Nov 17, 2021
…status #5358

Should have seen sooner that the API already proposes the upgrade to take so we just
make sure the related flag is set and read.
@fichtner
Copy link
Member Author

fichtner commented Nov 17, 2021

New flag status_reboot now exists with the correct reboot status for the advertised status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Low impact changes
Development

No branches or pull requests

1 participant