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

rdcore: Juggle physical root versus deployment root #1203

Merged
merged 2 commits into from
Nov 6, 2023

Conversation

cgwalters
Copy link
Member

rdcore: Juggle physical root versus deployment root

This is prep for supporting composefs, where the physical root
is distinct from the deployment root. Specifically for the LUKS
case, we can find /etc/crypttab only in the deployment root.

Otherwise, we suffix the passed path (usually /sysroot) that
was mounted in the initramfs with /sysroot to find the physical
root.

xref ostreedev/ostree#2867

@cgwalters
Copy link
Member Author

Any thoughts on this one?

@cgwalters
Copy link
Member Author

Ping

@cgwalters
Copy link
Member Author

I'd really like to turn on composefs by default soon for ostree, and this is going to block that.

The FCOS coreos-boot-edit.sh runs both of these, and it wasn't
immediately clear to me which of them was failing.
This is prep for supporting composefs, where the physical root
is distinct from the deployment root.  Specifically for the LUKS
case, we can find `/etc/crypttab` only in the deployment root.

Otherwise, we suffix the passed path (usually `/sysroot`) that
was mounted in the initramfs with `/sysroot` to find the physical
root.

xref ostreedev/ostree#2867
@jlebon jlebon force-pushed the composefs-rootmap branch from bc209f5 to 22d4f2b Compare November 6, 2023 21:24
@jlebon jlebon enabled auto-merge November 6, 2023 21:24
@jlebon jlebon merged commit f7a4e53 into coreos:main Nov 6, 2023
13 of 14 checks passed
@cgwalters
Copy link
Member Author

Looks like there hasn't been a release since this got merged? This is blocking composefs enablement in rawhide which I'd like to do soon.

cgwalters added a commit to cgwalters/fedora-coreos-config that referenced this pull request Dec 5, 2023
The goal is to start shaking out integration issues.

This currently blocks on a release of coreos-installer
with coreos/coreos-installer#1203
@jlebon
Copy link
Member

jlebon commented Dec 6, 2023

We should be cutting a release soon. I opened #1349.

cgwalters added a commit to cgwalters/fedora-coreos-config that referenced this pull request Jan 2, 2024
Prep for enabling composefs.  This is the same issue as
coreos/coreos-installer#1203
cgwalters added a commit to coreos/fedora-coreos-config that referenced this pull request Jan 3, 2024
Prep for enabling composefs.  This is the same issue as
coreos/coreos-installer#1203
cgwalters added a commit to cgwalters/fedora-coreos-config that referenced this pull request Jan 3, 2024
The goal is to start shaking out integration issues.

This currently blocks on a release of coreos-installer
with coreos/coreos-installer#1203
cgwalters added a commit to cgwalters/coreos-assembler that referenced this pull request Jan 3, 2024
Yet another instance of the same problem as coreos/coreos-installer#1203

Motivated by enabling composefs compatibility.
cgwalters added a commit to coreos/coreos-assembler that referenced this pull request Jan 3, 2024
Yet another instance of the same problem as coreos/coreos-installer#1203

Motivated by enabling composefs compatibility.
aaradhak pushed a commit to aaradhak/fedora-coreos-config that referenced this pull request Mar 18, 2024
Prep for enabling composefs.  This is the same issue as
coreos/coreos-installer#1203
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