Skip to content

fix(disk): increase /boot partition from 1 GiB to 2 GiB - #7

Open
castrojo wants to merge 1 commit into
devfrom
fix/2gib-boot-partition
Open

fix(disk): increase /boot partition from 1 GiB to 2 GiB#7
castrojo wants to merge 1 commit into
devfrom
fix/2gib-boot-partition

Conversation

@castrojo

Copy link
Copy Markdown

Each kernel+initramfs pair can be 200-400 MiB with large driver sets (e.g. NVIDIA). With a running deployment, a staged upgrade, and a rollback entry, 1 GiB fills up quickly.\n\n2 GiB provides comfortable headroom and matches the systemd-boot layout which already uses 2 GiB for the same reason.

Each kernel+initramfs pair can be 200-400 MiB with large driver sets
(e.g. NVIDIA). With a running deployment, a staged upgrade, and a
rollback entry, 1 GiB fills up quickly. 2 GiB provides comfortable
headroom and matches the systemd-boot layout which already uses 2 GiB.

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Clean 2-line change with solid rationale. CI failures are pre-existing nilerr/lvm issues unrelated to this PR.

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Clean 2-line change with solid rationale. CI failures are pre-existing nilerr/lvm issues unrelated to this PR. Ready to merge.

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Clean 2-line change with solid rationale. CI failures are pre-existing nilerr/lvm issues unrelated to this PR.

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: Changes requested

The partition size change (1 GiB → 2 GiB) is correct and well-motivated. However, the test TestPartition_SfdiskScript in partition_test.go still asserts size=1GiB and will fail with this change. Please update the test assertion to expect size=2GiB.

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.

2 participants