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-35994) kernel-install: fix uki-copy deinstall #271

Merged

Conversation

jacekmigacz
Copy link
Member

@jacekmigacz jacekmigacz commented May 15, 2024

For "kernel-install remove ..." only the kernel version is passed, not the kernel image. So auto-detecting KERNEL_INSTALL_IMAGE_TYPE and setting KERNEL_INSTALL_LAYOUT does not work for uninstall.

The 90-uki-copy.install plugin must consider this and not exit early for the "remove" command, otherwise $BOOT_ROOT will be filled with stale kernel images.

Signed-off-by: Gerd Hoffmann kraxel@redhat.com

(cherry picked from commit 3037616)

Resolves: RHEL-35994

For "kernel-install remove ..." only the kernel version is passed, not
the kernel image.  So auto-detecting KERNEL_INSTALL_IMAGE_TYPE and
setting KERNEL_INSTALL_LAYOUT does not work for uninstall.

The 90-uki-copy.install plugin must consider this and *not* exit early
for the "remove" command, otherwise $BOOT_ROOT will be filled with stale
kernel images.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>

(cherry picked from commit 3037616)

Resolves: RHEL-35994
@github-actions github-actions bot added rhel-9.4.0 pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels May 15, 2024
Copy link

github-actions bot commented May 15, 2024

Commit validation

Tracker - RHEL-35994

The following commits meet all requirements

commit upstream
ec9fca5 - kernel-install: fix uki-copy deinstall systemd/systemd@3037616

Tracker validation

Success

🟢 Tracker RHEL-35994 has set desired product: rhel-9.4
🟢 Tracker RHEL-35994 has set desired component: systemd
🟢 Tracker RHEL-35994 has been approved


Pull Request validation

Success

🟡 CI - Waived
🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved


Auto Merge

Failed

🔴 Pull Request has unsupported target branch rhel-9.4.0, expected branches are: 'main,master'

Success

🟢 Pull Request is not marked as draft and it's not blocked by dont-merge label
🟢 Pull Request meet requirements, title has correct form
🟢 Pull Request meet requirements, mergeable is true
🟠 Pull Request meet requirements, mergeable_state is unstable

Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot removed the pr/needs-review Formerly needs-review label May 16, 2024
@jamacku
Copy link
Member

jamacku commented May 17, 2024

Hmm, fuzzing failures are unrelated to changes from PR.

@github-actions github-actions bot added pr/needs-manual-merge Formerly pr/needs-manual-merge and removed pr/needs-ci Formerly needs-ci labels May 17, 2024
@jamacku jamacku self-assigned this May 17, 2024
@jamacku jamacku merged commit ba83f03 into redhat-plumbers:rhel-9.4.0 May 20, 2024
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants