Skip to content

nullbootctl: Sync the ESP and improve comments#139

Merged
mkukri-canonical merged 1 commit into
mainfrom
syncfs
Jun 25, 2026
Merged

nullbootctl: Sync the ESP and improve comments#139
mkukri-canonical merged 1 commit into
mainfrom
syncfs

Conversation

@julian-klode

Copy link
Copy Markdown
Contributor

Sync the ESP between copying in the kernels and adding them to the boot order so they are written to the backing storage before the boot order is updated.

This also ensures that when kernels are removed later, the new kernels are there so there always is a bootable path.

Also add additional documentation strings.

@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 18729579563

Details

  • 0 of 11 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.7%) to 71.012%

Changes Missing Coverage Covered Lines Changed/Added Lines %
cmd/nullbootctl/main.go 0 11 0.0%
Totals Coverage Status
Change from base Build 16876532719: -0.7%
Covered Lines: 779
Relevant Lines: 1097

💛 - Coveralls

@julian-klode

Copy link
Copy Markdown
Contributor Author

This needs testing and I need to update my signing key, GitHub doesn't seem to want to pull in new signatures itself and doesn't provide a way to update the key with news ones, sigh (key expiry is every 2 years I believe, and I resign them earlier)

Sync the ESP between copying in the kernels and adding
them to the boot order so they are written to the backing
storage before the boot order is updated.

This also ensures that when kernels are removed later,
the new kernels are there so there always is a bootable
path.

Also add additional documentation strings.
@mkukri-canonical
mkukri-canonical merged commit c35575e into main Jun 25, 2026
8 checks passed
@mkukri-canonical
mkukri-canonical deleted the syncfs branch June 25, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants