Skip to content

Conversation

@bmwiedemann
Copy link
Contributor

This helps a bit with issue #2358

Changes proposed in this pull request:

  • call dracut with --reproducible option to create reproducible initrd files.

@schaefi
Copy link
Collaborator

schaefi commented Jul 1, 2025

@bmwiedemann do we know what exactly this flag does behind the scene ?

If this is a safe change I have no objections. Can you please update the corresponding unit test too ?

Thanks much

@bmwiedemann
Copy link
Contributor Author

bmwiedemann commented Jul 1, 2025

From looking at
https://github.com/dracutdevs/dracut/blob/5d2bda46f4e75e85445ee4d3bd3f68bf966287b9/dracut.sh#L2301
it touches files+dirs to ensure fixed mtimes and calls cpio with --reproducible to not embed inode numbers into cpio headers.

It should be safe.

This helps a bit with issue OSInside#2358

Add reproducible flag for UKI too

Update tests accordingly

Co-Authored-By: Marcus Schäfer <[email protected]>
@bmwiedemann bmwiedemann force-pushed the reproducibledracut branch from 287928f to 4640c18 Compare July 1, 2025 09:31
@schaefi
Copy link
Collaborator

schaefi commented Jul 1, 2025

looks good, let's clarify if the change is compatible with the supported distro matrix. I will zip a Staging build and wait for the integration test runs. If all is good we are safe to merge

@schaefi
Copy link
Collaborator

schaefi commented Jul 1, 2025

All tests pass

@schaefi schaefi self-requested a review July 1, 2025 13:49
@schaefi schaefi merged commit 9a5a788 into OSInside:main Jul 1, 2025
13 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.

2 participants