-
Notifications
You must be signed in to change notification settings - Fork 562
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
o/snapstate: in hybrid systems re-refresh before reboot if possible #13955
Conversation
In hybrid systems where we do refreshes of apps before the reboot and essential snaps after, check if a re-refresh is necessary after the apps are done instead of waiting for all (and therefore a reboot). This is possible because currently essential snaps don't use epochs and don't require re-refreshes. If an app has a dependency that requires a reboot (like the model base), then the check-rerefresh task needs to wait for that refresh to complete. Since, if we're waiting for the reboot, there isn't much to gain in resolving the app's dependencies to only wait for those, the check-rerefresh preserves the old behavior in that case to keep things simple. Signed-off-by: Miguel Pires <miguel.pires@canonical.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, one question to consider.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you, some comments
Signed-off-by: Miguel Pires <miguel.pires@canonical.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you
In hybrid systems where we do refreshes of apps before the reboot and essential snaps after, check if a re-refresh is necessary after the apps are done instead of waiting for all (and therefore a reboot). This is possible because currently essential snaps don't use epochs and don't require re-refreshes. If an app has a dependency that requires a reboot (like the model base), then the check-rerefresh task needs to wait for that refresh to complete. Since, if we're waiting for the reboot, there isn't much to gain in resolving the app's dependencies to only wait for those, the check-rerefresh preserves the old behavior in that case to keep things simple.