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

Add support for isomd5sum for tagging iso files #2615

Merged
merged 1 commit into from
Aug 26, 2024
Merged

Conversation

Conan-Kudo
Copy link
Member

The isomd5sum tool suite is used and available on all supported distributions except SUSE distributions, and is necessary to produce conformant ISOs for most Linux distributions.

This change adds support for isomd5sum tool suite for kiwi, though it does not extend the kiwi-live dracut module to use it. The upstream dracut dmsquash-live module must be used instead.

@Conan-Kudo Conan-Kudo force-pushed the isomd5support branch 8 times, most recently from 0ab0e24 to 8be281e Compare August 19, 2024 09:48
@Conan-Kudo Conan-Kudo requested review from schaefi, davidcassany and dcermak and removed request for schaefi and davidcassany August 19, 2024 14:22
Copy link
Collaborator

@dcermak dcermak left a comment

Choose a reason for hiding this comment

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

I believe we have to change RuntimeChecker.check_mediacheck_installed, as I currently get this error:

[ ERROR   ]: 15:23:31 | KiwiRuntimeError: 

Required tool tagmedia not found in caller environment

The attribute 'mediacheck' is set to 'true' which requires
the above tool to be installed on the build system

kiwi/runtime_config.py Outdated Show resolved Hide resolved
kiwi/runtime_config.py Outdated Show resolved Hide resolved
@Conan-Kudo Conan-Kudo marked this pull request as draft August 22, 2024 12:44
@Conan-Kudo Conan-Kudo force-pushed the isomd5support branch 7 times, most recently from 9b509af to d89c0cf Compare August 24, 2024 23:26
@Conan-Kudo
Copy link
Member Author

Tests pass again, but now coverage fails... 😦

/home/runner/work/kiwi/kiwi/kiwi/runtime_checker.py                                333      2    99%   924, 926
/home/runner/work/kiwi/kiwi/kiwi/runtime_config.py                                 110      3    97%   332, 336-341

@Conan-Kudo
Copy link
Member Author

@schaefi, @davidcassany: either of you able to help me figure out how to close this coverage gap so this becomes green?

@dcermak
Copy link
Collaborator

dcermak commented Aug 26, 2024

@schaefi, @davidcassany: either of you able to help me figure out how to close this coverage gap so this becomes green?

The mocks were not setup correctly, I have force pushed a fix for runtime_checker_test

The isomd5sum tool suite is used and available on all supported
distributions except SUSE distributions, and is necessary to produce
conformant ISOs for most Linux distributions.

This change adds support for isomd5sum tool suite for kiwi, though
it does not extend the kiwi-live dracut module to use it. The upstream
dracut dmsquash-live module must be used instead.

Co-authored-by: Dan Čermák <[email protected]>
Copy link
Collaborator

@dcermak dcermak left a comment

Choose a reason for hiding this comment

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

I think this should work now, at least a test build doesn't die immediately.

Does the documentation need updating somewhere?

@Conan-Kudo
Copy link
Member Author

I think this should work now, at least a test build doesn't die immediately.

Does the documentation need updating somewhere?

Nope. The documentation for the kiwi.yml file is in the kiwi.yml file itself.

@Conan-Kudo Conan-Kudo marked this pull request as ready for review August 26, 2024 10:55
@Conan-Kudo Conan-Kudo merged commit aa908ae into main Aug 26, 2024
12 checks passed
@Conan-Kudo Conan-Kudo deleted the isomd5support branch August 26, 2024 11:26
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.

2 participants