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

dracut: Create reproducible images #2170

Merged
merged 1 commit into from
Aug 16, 2020

Conversation

jprvita
Copy link
Contributor

@jprvita jprvita commented Aug 13, 2020

Without reproducible images, a rebuild of the initrd will create a
different image file (due to things like creation time of the files in
the cpio archive) even if the actual contents in it are exactly the
same, adding an unnecessary download during updates.

Adding 'reproducible=yes' avoids this and creates the same image files
for the same content.

Without reproducible images, a rebuild of the initrd will create a
different image file (due to things like creation time of the files in
the cpio archive) even if the actual contents in it are exactly the
same, adding an unnecessary download during updates.

Adding 'reproducible=yes' avoids this and creates the same image files
for the same content.
@openshift-ci-robot
Copy link
Collaborator

Hi @jprvita. Thanks for your PR.

I'm waiting for a ostreedev member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jprvita
Copy link
Contributor Author

jprvita commented Aug 13, 2020

We are shipping this change on Endless OS downstream.

@jlebon
Copy link
Member

jlebon commented Aug 13, 2020

Wouldn't it be cleaner to have whatever runs dracut ensure it passes --reproducible? This is what rpm-ostree does (and actually, Fedora/RHEL doesn't ship this config at all nowadays; see https://bugzilla.redhat.com/show_bug.cgi?id=1331369).

Realistically, this is probably fine and I agree that most (all?) users of ostree + dracut would want this. Though it feels like this is at the wrong level of the stack; given that --reproducible affects the whole initramfs ISTM this belongs in the layer that binds ostree and dracut together.

Anyway, weak
/approve

Will let @cgwalters have a look.

@cgwalters
Copy link
Member

Agree with @jlebon - this is a distro decision, not ostree's decision (though it clearly benefits ostree-based systems a lot).

I also think basically everyone should be using --with-dracut=yesbutnoconf nowadays as Fedora is - the config file we install is equivalent to just having --add=ostree on the dracut cmdline, which is utterly trivial to do (and is also what rpm-ostree does).

But that said, this doesn't hurt so
/lgtm

@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, jlebon, jprvita, mwleeds

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot merged commit 83bb09a into ostreedev:master Aug 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants