Skip to content

Fix creation of checksums in customize workflow#302

Merged
dirkmueller merged 3 commits intoSUSE:mainfrom
dirkmueller:checksumming_fix
Dec 11, 2025
Merged

Fix creation of checksums in customize workflow#302
dirkmueller merged 3 commits intoSUSE:mainfrom
dirkmueller:checksumming_fix

Conversation

@dirkmueller
Copy link
Copy Markdown
Member

No description provided.

@dirkmueller dirkmueller requested a review from a team as a code owner December 11, 2025 10:13
@dirkmueller dirkmueller marked this pull request as draft December 11, 2025 10:19
Comment thread internal/customize/customize.go Outdated
@dirkmueller dirkmueller force-pushed the checksumming_fix branch 5 times, most recently from feb6296 to 580f0f4 Compare December 11, 2025 12:00
@dirkmueller dirkmueller marked this pull request as ready for review December 11, 2025 12:08
Copy link
Copy Markdown
Contributor

@davidcassany davidcassany left a comment

Choose a reason for hiding this comment

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

Nice! Just a couple of small comments. Feel free to ignore them or leave it for later as it can be rearranged when addressing the TODO.

Comment thread internal/image/install/install.go
Comment thread pkg/installer/media.go Outdated
Cannot use go-units FromHumanSize() because we do not follow SI standard
in our API. 10G == 10 GiB
The previous path first calculated the checksum and then truncated
which led to mismatching checksum. Increase default size to 12G
Comment thread internal/customize/customize.go Outdated
@dirkmueller dirkmueller merged commit 27aa11f into SUSE:main Dec 11, 2025
4 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.

4 participants