Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

Enable composefs root #356

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Conversation

cgwalters
Copy link
Member

c9s: Pull in latest bootc/ostree

On general principle, but specifically to enable transient root.


Enable transient (composefs) root (again)

This reverts commit 7977ead and
effectively migrates the change from CentOS/centos-bootc-dev@8f5be09
to here.

See the linked PR for much more debate, but in a nutshell this
flips around a lot of tradeoffs around the filesystem layout
and will result in a much more easily explained state that
is intended to somewhat more closely match what happens with
a default podman run --rm invocation of the container.

Signed-off-by: Colin Walters [email protected]


@cgwalters
Copy link
Member Author

Blocks on CentOS/centos-bootc-dev#30

@cgwalters
Copy link
Member Author

@cgwalters
Copy link
Member Author

And now it blocks on a development compose being spun, which I think is hopefully tomorrow

@cgwalters
Copy link
Member Author

This will work on top of #381

@cgwalters cgwalters force-pushed the enable-composefs-transient branch from cce71d0 to b0645e2 Compare March 1, 2024 14:10
@cgwalters cgwalters changed the title Enable transient (composefs) root (again) Enable composefs root Mar 1, 2024
@cgwalters cgwalters marked this pull request as ready for review March 1, 2024 15:42
@cgwalters cgwalters force-pushed the enable-composefs-transient branch 2 times, most recently from 707b3fe to 82a8daa Compare March 1, 2024 18:43
This partially reverts commit 7977ead and
effectively migrates the change from CentOS/centos-bootc-dev@8f5be09
and CentOS/centos-bootc-dev@14ab1c5
to here.

Signed-off-by: Colin Walters <[email protected]>
@cgwalters cgwalters force-pushed the enable-composefs-transient branch from 82a8daa to 2257dd6 Compare March 1, 2024 18:52
@cgwalters cgwalters enabled auto-merge March 1, 2024 18:52
Copy link
Collaborator

@sallyom sallyom left a comment

Choose a reason for hiding this comment

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

LGTM!

@cgwalters cgwalters merged commit e49fe2b into CentOS:main Mar 1, 2024
4 of 6 checks passed
@cgwalters

This comment was marked as resolved.

@cgwalters
Copy link
Member Author

I was wrong above, I apparently had some local pilot error, upgrades work just fine.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants