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

pkg/mkimage-raw-efi,pkg/mkverification-raw-efi: Rename 003-installer to 002-installer #3913

Merged

Conversation

rouming
Copy link
Contributor

@rouming rouming commented May 10, 2024

In the b723823 ("rootfs: move modprobe to the 000-mod-params initd script") the number of the onboot containers was changed and 003-installer became 002-installer (modprobe was moved to the corresponding init script). This container rename was not fully reflected in all the files, which do the linuxkit magic and still access the 003-installer directly.

This patch does the s/003-installer/002-installer/.

Fixes: b723823 ("rootfs: move modprobe to the 000-mod-params initd script")

CC: @zedi-pramodh

…to 002-installer

In the b723823 ("rootfs: move modprobe to the 000-mod-params initd script")
the number of the onboot containers was changed and 003-installer became 002-installer
(modprobe was moved to the corresponding init script). This container rename was not
fully reflected in all the files, which do the linuxkit magic and still access the
003-installer directly.

This patch does the s/003-installer/002-installer/.

Fixes: b723823 ("rootfs: move modprobe to the 000-mod-params initd script")
Signed-off-by: Roman Penyaev <r.peniaev@gmail.com>
@zedi-pramodh
Copy link

Tested this fix works on kubevirt eve

Copy link

@zedi-pramodh zedi-pramodh left a comment

Choose a reason for hiding this comment

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

LGTM

@rouming rouming merged commit 7d6db4d into lf-edge:master May 13, 2024
37 of 43 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.

None yet

3 participants