docs: bootloader-configuration by bfjelds · Pull Request #256 · microsoft/trident

bfjelds · 2025-10-09T18:47:21Z

No description provided.

Copilot

Pull Request Overview

This PR adds comprehensive documentation for bootloader configuration in Trident, replacing a placeholder document with detailed explanation of how Trident handles GRUB and systemd-boot bootloaders.

Explains COSI configuration requirements for different bootloader types
Documents bootloader servicing process including A/B update handling
Details systemd-boot specific naming conventions and file management

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

ayaegashi · 2025-10-09T20:20:44Z

iirc bootloader configuration is closely related to encryption. Wondering if there would be a useful snippet to include about encryption in this doc, @ndubchak

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

bfjelds · 2025-10-09T20:38:10Z

iirc bootloader configuration is closely related to encryption. Wondering if there would be a useful snippet to include about encryption in this doc, @ndubchak

added a note that references encryption api-reference pcr section.

Britel

Left some small suggestions. Otherwise good to go

Co-authored-by: Brian Telfer <britel@microsoft.com>

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-10-14T20:09:30Z

+illustrate how this process works.
+:::
+
+In order to support A/B update from a single ESP partition, Trident needs to


Grammatical number issue: change 'A/B update' to 'A/B updates' (general capability) for clarity.

Suggested change

In order to support A/B update from a single ESP partition, Trident needs to

In order to support A/B updates from a single ESP partition, Trident needs to

Copilot · 2025-10-14T20:09:31Z

+* `/boot/efi/EFI/AZLA` - the target OS for the initial install
+* `/boot/efi/EFI/AZLB` - the target OS for a future update
+
+Within the bootloader paths, Trident will copy efi files (like `boot<ARCH>.efi`


Acronym 'EFI' should be capitalized for consistency with earlier usage (e.g., 'EFI/AZLA').

Suggested change

Within the bootloader paths, Trident will copy efi files (like `boot<ARCH>.efi`

Within the bootloader paths, Trident will copy EFI files (like `boot<ARCH>.efi`

Copilot · 2025-10-14T20:09:31Z

+it is versioned with the kernel version (for example
+`/boot/efi/EFI/Linux/vmlinuz-6.6.96.2-2.azl3.efi`), to
+[`/boot/efi/EFI/Linux`](https://uapi-group.org/specifications/specs/boot_loader_specification/#locating-boot-entries)
+on the target OS. Trident will rename the UKI efi file to ensure that the


Capitalize the acronym 'EFI' for consistency ('UKI EFI file').

Suggested change

on the target OS. Trident will rename the UKI efi file to ensure that the

on the target OS. Trident will rename the UKI EFI file to ensure that the

Copilot · 2025-10-14T20:09:31Z

+
+For example, the first install of a single OS would create
+`/boot/efi/EFI/Linux/vmlinuz-100-azla0.efi`. A subsequent update would create
+`/boot/efi/EFI/Linux/vmlinuz-101-azlb0.efi`. A subsequent update would create


[nitpick] The phrase 'A subsequent update would create' is repeated on consecutive lines, which can be confusing. Consider rephrasing the second occurrence to 'The next update would create' or enumerate the sequence.

Suggested change

`/boot/efi/EFI/Linux/vmlinuz-101-azlb0.efi`. A subsequent update would create

`/boot/efi/EFI/Linux/vmlinuz-101-azlb0.efi`. The next update would create

init

4d8bc4f

bfjelds requested a review from a team as a code owner October 9, 2025 18:47

Copilot AI review requested due to automatic review settings October 9, 2025 18:47

Copilot AI reviewed Oct 9, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

Update docs/Explanation/Bootloader-Configuration.md

d651bcb

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings October 9, 2025 18:49

Copilot AI reviewed Oct 9, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

bfjelds added 2 commits October 9, 2025 18:57

more

c4321b0

ic docs

bcc31fb

Copilot AI review requested due to automatic review settings October 9, 2025 18:59

Copilot AI reviewed Oct 9, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

Update docs/Explanation/Bootloader-Configuration.md

b9a1c72

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings October 9, 2025 19:11

Copilot AI reviewed Oct 9, 2025

View reviewed changes

two spaces

3904554

uki/grub vs encryption

fe346a9

Copilot AI review requested due to automatic review settings October 9, 2025 20:37

Copilot AI reviewed Oct 9, 2025

View reviewed changes

Britel reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

Britel reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

Britel reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

Britel previously approved these changes Oct 10, 2025

View reviewed changes

Update docs/Explanation/Bootloader-Configuration.md

89d4469

Co-authored-by: Brian Telfer <britel@microsoft.com>

bfjelds dismissed Britel’s stale review via 89d4469 October 10, 2025 14:47

Copilot AI review requested due to automatic review settings October 10, 2025 14:47

Update docs/Explanation/Bootloader-Configuration.md

0be82f5

Co-authored-by: Brian Telfer <britel@microsoft.com>

Copilot AI reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md

feedback

1727e27

ndubchak reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

ndubchak reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md

ndubchak reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

ndubchak reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

fix link

2e130be

Copilot AI review requested due to automatic review settings October 10, 2025 21:50

ndubchak reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md

Copilot AI reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

ndubchak reviewed Oct 10, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md

bfjelds and others added 2 commits October 10, 2025 21:52

fix

814c0bc

Update docs/Explanation/Bootloader-Configuration.md

e38abc5

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings October 10, 2025 21:57

Copilot AI reviewed Oct 10, 2025

View reviewed changes

bfjelds added 2 commits October 10, 2025 22:02

add note

e84a202

target os

7901b72

Copilot AI review requested due to automatic review settings October 10, 2025 22:13

Copilot AI reviewed Oct 10, 2025

View reviewed changes

uki-systemd relationship

b873368

ndubchak reviewed Oct 13, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md

ndubchak reviewed Oct 13, 2025

View reviewed changes

Comment thread docs/Explanation/Bootloader-Configuration.md Outdated

EFI

cabb7c0

Copilot AI review requested due to automatic review settings October 14, 2025 00:16

Copilot AI reviewed Oct 14, 2025

View reviewed changes

bfjelds added 2 commits October 14, 2025 19:40

shim

aa2cf0e

grub and systemd

cff807c

Copilot AI review requested due to automatic review settings October 14, 2025 20:08

Copilot AI reviewed Oct 14, 2025

View reviewed changes

ndubchak approved these changes Oct 14, 2025

View reviewed changes

bfjelds merged commit 065568d into main Oct 14, 2025
11 checks passed

bfjelds deleted the user/bfjelds/bootloader-configuration branch February 13, 2026 02:23

	In order to support A/B update from a single ESP partition, Trident needs to
	In order to support A/B updates from a single ESP partition, Trident needs to

	Within the bootloader paths, Trident will copy efi files (like `boot<ARCH>.efi`
	Within the bootloader paths, Trident will copy EFI files (like `boot<ARCH>.efi`

	on the target OS. Trident will rename the UKI efi file to ensure that the
	on the target OS. Trident will rename the UKI EFI file to ensure that the

	`/boot/efi/EFI/Linux/vmlinuz-101-azlb0.efi`. A subsequent update would create
	`/boot/efi/EFI/Linux/vmlinuz-101-azlb0.efi`. The next update would create

Conversation

bfjelds commented Oct 9, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

ayaegashi commented Oct 9, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

bfjelds commented Oct 9, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Britel left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

ndubchak Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!