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

RHEL-7999: distro/rhel9: Make /boot 600 MiB big on RHEL 9.3+ #186

Merged
merged 2 commits into from
Oct 5, 2023

Conversation

ondrejbudai
Copy link
Member

Changes in xfsprogs in 9.3 caused XFS partitions have less available space (due to more space reserved for the log section).

We discovered that this causes issues with kernel updates. When 3 kernels are already installed and dnf wants to install a new one (and replace the oldest one), /boot must contain 4 kernels for a brief period of time, which doesn't fit on the small partition. This causes the transaction test to fail, and dnf refuses to perform such a transaction.

There's a workaround: Remove one of the kernels manually using dnf remove before running an update.

This is however quite surprising and annoying, thus this commits bumps the size of /boot to 600 MiB starting from 9.3 (and on CentOS Stream).

In order to implement this change, I had to make the base partition table a function. This feels a bit ugly, but I couldn't think of a cleaner way without much duplicated code.

In the next commit, we will want to make adjustments based on the RHEL
minor version.
@ondrejbudai ondrejbudai added the WIP+test Work in progress but run Gitlab CI. label Oct 4, 2023
Changes in xfsprogs in 9.3 caused XFS partitions have less available space
(due to more space reserved for the log section).

We discovered that this causes issues with kernel updates. When 3 kernels are
already installed and dnf wants to install a new one (and replace the oldest
one), /boot must contain 4 kernels for a brief period of time, which
doesn't fit on the small partition. This causes the transation test to fail,
and dnf refuses to perform such a transaction.

There's a workaround: Remove one of the kernels manually using `dnf remove`
before running an update.

This is however quite surprising and annoying, thus this commits bumps the
size of /boot to 600 MiB starting from 9.3 (and on CentOS Stream).
@ondrejbudai ondrejbudai changed the title distro/rhel9: Make /boot 600 MiB big on RHEL 9.3+ RHEL-7999: distro/rhel9: Make /boot 600 MiB big on RHEL 9.3+ Oct 4, 2023
@ondrejbudai ondrejbudai removed the WIP+test Work in progress but run Gitlab CI. label Oct 5, 2023
@ondrejbudai ondrejbudai marked this pull request as ready for review October 5, 2023 12:57
Copy link
Member

@thozza thozza left a comment

Choose a reason for hiding this comment

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

LGTM

@achilleas-k achilleas-k added this pull request to the merge queue Oct 5, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 5, 2023
@ondrejbudai ondrejbudai added this pull request to the merge queue Oct 5, 2023
Merged via the queue into osbuild:main with commit c7925a2 Oct 5, 2023
9 checks passed
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