Skip to content

spec: backport --entry-type support to kernel-install preun - #1481

Merged
bmastbergen merged 1 commit into
ciq-6.12.yfrom
ciq-6.12.y-uki-fix-v2
Jul 29, 2026
Merged

spec: backport --entry-type support to kernel-install preun#1481
bmastbergen merged 1 commit into
ciq-6.12.yfrom
ciq-6.12.y-uki-fix-v2

Conversation

@bmastbergen

Copy link
Copy Markdown
Collaborator

Without --entry-type, uninstalling the UKI package runs kernel-install remove which removes all boot entries for that kernel version, including the Type #1 BLS entry belonging to kernel-core. This was fixed upstream in the 6.18 spec.

Backport the fix: add -e flag support to the kernel_variant_preun macro which, when kernel-install supports --entry-type, passes --entry-type type1 for core packages and --entry-type type2 for UKI packages. A runtime probe ensures graceful degradation on older systemd versions.

Supersedes #1463 (rebased onto updated ciq-6.12.y).

Without --entry-type, uninstalling the UKI package runs
`kernel-install remove` which removes all boot entries for that kernel
version, including the Type #1 BLS entry belonging to kernel-core.
This was fixed upstream in the 6.18 spec.
Backport the fix: add -e flag support to the kernel_variant_preun macro
which, when kernel-install supports --entry-type, passes --entry-type
type1 for core packages and --entry-type type2 for UKI packages. A
runtime probe ensures graceful degradation on older systemd versions.
Signed-off-by: Brett Mastbergen <bmastbergen@ciq.com>
@bmastbergen
bmastbergen requested a review from a teamJuly 28, 2026 18:07

@PlaidCatPlaidCat left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

:shipit:

@PlaidCat
PlaidCat requested a review from a teamJuly 28, 2026 18:19
@bmastbergen
bmastbergen merged commit af6d9d8 into ciq-6.12.yJul 29, 2026
8 checks passed
@bmastbergen
bmastbergen deleted the ciq-6.12.y-uki-fix-v2 branch July 29, 2026 14:05
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@bmastbergen@PlaidCat@shreeya-patel98