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

Enable composefs for 41+ #3058

Closed

Conversation

jbtrystram
Copy link
Contributor

We really want to aim to eventually enable this by default, let's
test things out in rawhide.

A thing that this is known to break is the "chattr -i" hack
for new toplevel dirs (xref coreos/rpm-ostree#337 )

Basically if you want that, you either need to make a derived image,
or enable transient root.

Replaces #3009

We really want to aim to eventually enable this by default, let's
test things out in rawhide.

A thing that this is known to break is the "chattr -i" hack
for new toplevel dirs (xref coreos/rpm-ostree#337 )

Basically if you want that, you either need to make a derived image,
or enable transient root.
@jbtrystram
Copy link
Contributor Author

The second commit is taken from issues.redhat.com/browse/RHEL-35885
as this PR had issues with kdump

@jbtrystram jbtrystram marked this pull request as draft July 10, 2024 16:03
@jlebon
Copy link
Member

jlebon commented Jul 10, 2024

Replaces #3009

Hmm, the goal of that PR was to test the changes against rawhide in CI. Here, CI will not test this since it runs against testing-devel. Did you locally test a rawhide build passes kola with this?

@jbtrystram
Copy link
Contributor Author

Hmm, the goal of that PR was to test the changes against rawhide in CI. Here, CI will not test this since it runs against testing-devel. Did you locally test a rawhide build passes kola with this?

Ah yes indeed, it's been a while since i updated this and I saw that kdump fix I wanted to try it. The manifest is conditionally included on rawhide though, does that achieve the same goal ?
Ci passes on #3009 : https://jenkins-coreos-ci.apps.ocp.fedoraproject.org/blue/organizations/jenkins/fedora-coreos-config/detail/PR-3058/3/pipeline/

@jlebon
Copy link
Member

jlebon commented Jul 15, 2024

@jbtrystram jbtrystram closed this Jul 23, 2024
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.

3 participants