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

Add support for update --apply #165

Closed
cgwalters opened this issue Oct 31, 2023 · 0 comments · Fixed by #173
Closed

Add support for update --apply #165

cgwalters opened this issue Oct 31, 2023 · 0 comments · Fixed by #173
Labels
enhancement New feature or request

Comments

@cgwalters
Copy link
Collaborator

Today we have e.g. rpm-ostree upgrade --reboot. bootc update does not have an equivalent verb.

In this proposal we add update --apply which would today by default reboot. In the future ostree should gain support for systemd soft-reboot.

A bit longer term than that, we'll want to have rich tooling for admins/tooling more directly controlling the flow here.

First, an important detail is that I think we should default to locking finalization so that a reboot for other reasons doesn't apply the update - this ensures predictability for the sysadmin. This would be exposed as a new locked: bool in the CRD for staged deployments that could be changed to false if the admin/upgrade tooling wanted a reboot to also apply by default.

Also, what I think would be helpful here is to add some sort of "diff" to the CRD as well that says what the "apply strategy" would be (reboot, soft reboot or live apply).

@cgwalters cgwalters added the enhancement New feature or request label Oct 31, 2023
cgwalters added a commit to cgwalters/bootc that referenced this issue Nov 2, 2023
Closes: containers#165

Signed-off-by: Colin Walters <[email protected]>
cgwalters added a commit to cgwalters/bootc that referenced this issue Nov 2, 2023
Closes: containers#165

Signed-off-by: Colin Walters <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant