Skip to content

Conversation

@schaefi
Copy link
Collaborator

@schaefi schaefi commented Aug 5, 2025

In case the kiwi-repart module is explicitly requested in a dracut.conf file and the image is also configured to build an install ISO image this leads the install ISO to contain the kiwi-repart module as well which is unwanted. This commit explicitly omits the kiwi-repart when creating the initrd for the install image

In case the kiwi-repart module is explicitly requested in a
dracut.conf file and the image is also configured to build an
install ISO image this leads the install ISO to contain the
kiwi-repart module as well which is unwanted. This commit
explicitly omits the kiwi-repart when creating the initrd
for the install image
@schaefi schaefi requested a review from Conan-Kudo August 5, 2025 14:07
@schaefi schaefi self-assigned this Aug 5, 2025
@schaefi
Copy link
Collaborator Author

schaefi commented Aug 5, 2025

needs verification from @uecasm

@Conan-Kudo Conan-Kudo enabled auto-merge August 5, 2025 14:08
@Conan-Kudo Conan-Kudo disabled auto-merge August 5, 2025 14:08
There was a method named initialize defined and implemented
differently in the dracut modules kiwi-lib and kiwi-repart.
kiwi-lib is expected to be shared code across all kiwi dracut
modules. However if one module redefines a method of the
same name which is used in another module and expected to
work differently there, this is evil. This commit cleans
up the name conflict and names the kiwi library init function
as lib_initialize. All dracut code that is expected to make
use of this method has been adopted too.
@schaefi
Copy link
Collaborator Author

schaefi commented Aug 8, 2025

@uecasm Added a commit to also fix the name conflict. Can you please check if that makes sense ? Thanks much

@schaefi schaefi merged commit 0f756f3 into main Aug 11, 2025
14 checks passed
@schaefi schaefi deleted the omit_kiwi_repart_for_affected_builders branch August 11, 2025 15:38
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