-
Notifications
You must be signed in to change notification settings - Fork 54
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 ability to use plain squashfs as the iso rootfs #1105
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks! 😍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks sane to me but I will do my usual thing that I think the OS content should drive partitioning.
Moving this back to draft for the moment, I need to move RootfsType into manifest instead of image to make it easier to add Erofs support later (it will need to switch out the Squashfs stage in anaconda_installer_iso_tree.go, I'll do that Monday. |
Fedora 41 and later, and RHEL 10 use plain squashfs instead of compressing an ext4 filesystem.
This test assumed the image's static list of pipeline names must match the manifest, but it really only needs to check the ones with rpm or ostree metadata. To support the ability to have images with different pipelines (eg. squashfs only vs. ext4+squashfs rootfs iso creation) this changes the test to make sure the pipelines in the static list are in the manifest, and in the order specified. But the two lists do not need to match exactly.
This pipeline is no longer used on Fedora 41 and later. They use a plain squashfs install.img instead of wrapping an ext4 filesystem with squashfs. But it may be present in older versions, the test in distro_test.go has been adjusted to ensure that all pipelines named are present and in the correct order without requiring that this is an exact list of pipeline names.
Make sure that the rootfs-image pipeline is present in ext4 images and not present in squashfs only images.
Ok, this should be ready now. There followup to this later that will add erofs support instead of squashfs, but that's for later. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cheers! This all LGTM.
No description provided.