-
Notifications
You must be signed in to change notification settings - Fork 147
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
Late commands vs done #2013
Late commands vs done #2013
Conversation
2fc7899
to
46c857e
Compare
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.
I would like to separate the removal of the unattended-upgrades cancellation from the late-commands status change. I will split the commits into two accordingly.
Today, late-commands can in some cases not actually run, if the application reports a DONE state and a reboot is permitted. We now add LATE_COMMANDS to track this explicit state - we no longer transition to a DONE state while these commands are running.
Last cycle, to make the installation faster, we decided to disable the unattended-upgrades minimal steps. This prevents proper handling of unattended-upgrades cancellation so we now drop the feature completely. Furthermore, the status reported on the text installer UI is misleading when unattended-upgrades is running.
46c857e
to
a39fb03
Compare
NOTE: This was related to how I injected the new subiquity code into an existing ubuntu-desktop-bootstrap snap without regenerating the types. Dennis confirmed that the fix was working after regenerating the types. |
…ommand Since we removed the ability to interrupt the unattended-upgrades process, we can use run_curtin_command instead of start_curtin_command. Signed-off-by: Olivier Gayot <olivier.gayot@canonical.com>
94f6e6c
to
ae581d9
Compare
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.
LGTM. I can't Approve since I opened the PR ;)
No description provided.