-
Notifications
You must be signed in to change notification settings - Fork 231
Add systemd-boot bootchooser #1367
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
base: master
Are you sure you want to change the base?
Conversation
a5fa810
to
021b534
Compare
} | ||
|
||
/* Set the default boot choice to this slot name */ | ||
/* TODO: allow the slot name to be configurable */ |
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.
Should add a config option for this.
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.
Shouldn't this be the slot's bootname?
Add some really basic support to set the EFI variable for systemd-boot's default boot choice EFI variable. This is currently set when `rauc status mark-good` is called; meaning that once the system has started successfully the default boot choice EFI variable will be updated to reflect from the next boot onwards. Signed-off-by: Christopher Obbard <chris.obbard@collabora.com>
021b534
to
6c89078
Compare
Codecov ReportAttention: Patch coverage is
❌ Your patch check has failed because the patch coverage (16.00%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## master #1367 +/- ##
==========================================
- Coverage 79.95% 79.87% -0.08%
==========================================
Files 67 67
Lines 20057 20082 +25
==========================================
+ Hits 16037 16041 +4
- Misses 4020 4041 +21 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
How would this interact with https://systemd.io/AUTOMATIC_BOOT_ASSESSMENT/, which is also implemented by systemd-boot? |
This would need to be rebased, as #1636 has split the bootloader backends into different files. |
Add some really basic support to set the EFI variable for systemd-boot's default boot choice EFI variable.
This is currently set when "rauc status mark-good" is called; meaning that once the system has started successfully the default boot choice EFI variable will be updated to reflect from the next boot onwards.
This PR needs (at least):
r_boot_get_state
,r_boot_get_primary
,r_boot_set_primary
For context, this is from a project which we converted to use
systemd-boot
for its bootloader and is working quite well. I am creating a draft PR to see if there is any interest in accepting this into upstream, I can do all of the work to get it in.