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

Remove writable-paths #74

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

valentindavid
Copy link
Collaborator

@valentindavid valentindavid commented Feb 9, 2022

Instead we expect to load mounts from /etc/fstab from core.

/sysroot /sysroot/writable are the only mountpoints that need to be
mounted before /etc/fstab loaded, so only those mount units are left.
The rest will be automatically loaded.

Before loading /etc/fstab we need to prepare the writable disk. So
before initrd-root-fs.target and after /sysroot/writable is mounted,
we call systemd-tmpfiles for core-writable.conf.

This commit requires change in core-base to provide
/etc/fstab and core-writable.conf.

See canonical/core-base#25

@valentindavid valentindavid force-pushed the valentindavid/remove-writable-paths branch from 811f98f to 34e7f2a Compare February 22, 2022 15:00
@anonymouse64 anonymouse64 self-requested a review February 22, 2022 18:00
@valentindavid valentindavid force-pushed the valentindavid/remove-writable-paths branch 3 times, most recently from c6c51ca to dbdf3be Compare March 10, 2022 14:31
@pedronis
Copy link
Collaborator

As we hinted/discussed and after talking with @mvo5 again, given the timing, at this point this change should be scheduled for UC 24, to be done early in its development

Copy link
Contributor

@anonymouse64 anonymouse64 left a comment

Choose a reason for hiding this comment

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

the approach here has my +1 but I unfortunately won't have time to review the details

Instead we expect to load mounts from /etc/fstab from core.

/sysroot /sysroot/writable are the only mountpoints that need to be
mounted before /etc/fstab loaded, so only those mount units are left.
The rest will be automatically loaded.

Before loading /etc/fstab we need to prepare the writable disk. So
before initrd-root-fs.target and after /sysroot/writable is mounted,
we call systemd-tmpfiles for core-writable.conf.

This commit requires change in core-base to provide
/etc/fstab and core-writable.conf.
@valentindavid valentindavid force-pushed the valentindavid/remove-writable-paths branch from dbdf3be to aa0f1c4 Compare November 14, 2022 15:45
@pedronis pedronis self-requested a review November 16, 2022 12:35
@alfonsosanchezbeato alfonsosanchezbeato marked this pull request as draft May 1, 2023 13:24
@xnox xnox added the core-main Ubuntu Core Main label Sep 15, 2023
@xnox
Copy link
Contributor

xnox commented Jan 23, 2024

And here we are in UC24. Can we do it? is it already done elsehow?

@pedronis
Copy link
Collaborator

@xnox

And here we are in UC24. Can we do it? is it already done elsehow?

we didn't do it early in the development, so not yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked core-main Ubuntu Core Main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants