Merge ~anonymouse67/ubuntu-core-initramfs:feature/uc20-run-mnt-data into ~canonical-foundations/ubuntu-core-initramfs:master

Proposed by Ian Johnson
Status: Merged
Merged at revision: c47b34414ff7e76b0d4b9fc145158108299bfeb6
Proposed branch: ~anonymouse67/ubuntu-core-initramfs:feature/uc20-run-mnt-data
Merge into: ~canonical-foundations/ubuntu-core-initramfs:master
Diff against target: 58 lines (+6/-6)
4 files modified
dev/null (+0/-1)
factory/usr/lib/systemd/system/populate-writable.service (+2/-2)
factory/usr/lib/systemd/system/run-mnt-data.mount.wants/sysroot-writable.mount (+1/-0)
factory/usr/lib/systemd/system/sysroot-writable.mount (+3/-3)
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Michael Vogt (community) Approve
Dimitri John Ledkov Pending
Review via email: mp+383515@code.launchpad.net

Description of the change

This is necessary to stop using /run/mnt/ubuntu-data from snap-bootstrap and everywhere really as per the new design on the names.

The associated snapd PR has yet to be merged but is dependent on this MP, see https://github.com/snapcore/snapd/pull/8464

To post a comment you must log in.
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Is this ready to be merged now? or doesn't need to be after PR8464?

What's the ordering sequence here?

Revision history for this message
Ian Johnson (anonymouse67) wrote :

This needs to be merged after https://github.com/snapcore/snapd/pull/8610 is merged and released. Michael said he would take care of that in his AM on Thursday

Revision history for this message
Michael Vogt (mvo) wrote :

This is ready to be merged now.

Revision history for this message
Michael Vogt (mvo) wrote :

LGTM

review: Approve
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Ok.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/factory/usr/lib/systemd/system/populate-writable.service b/factory/usr/lib/systemd/system/populate-writable.service
2index 30d9ae3..c914015 100644
3--- a/factory/usr/lib/systemd/system/populate-writable.service
4+++ b/factory/usr/lib/systemd/system/populate-writable.service
5@@ -3,12 +3,12 @@ OnFailure=emergency.target
6 OnFailureJobMode=replace-irreversibly
7 Before=initrd-cleanup.target
8 After=run-mnt-ubuntu\x2dseed.mount
9-After=run-mnt-ubuntu\x2ddata.mount
10+After=run-mnt-data.mount
11 After=run-mnt-base.mount
12 After=sysroot.mount
13 After=sysroot-writable.mount
14 Requires=run-mnt-ubuntu\x2dseed.mount
15-Requires=run-mnt-ubuntu\x2ddata.mount
16+Requires=run-mnt-data.mount
17 Requires=run-mnt-base.mount
18 Requires=sysroot.mount
19 Requires=sysroot-writable.mount
20diff --git a/factory/usr/lib/systemd/system/run-mnt-data.mount.wants/sysroot-writable.mount b/factory/usr/lib/systemd/system/run-mnt-data.mount.wants/sysroot-writable.mount
21new file mode 120000
22index 0000000..0d3b832
23--- /dev/null
24+++ b/factory/usr/lib/systemd/system/run-mnt-data.mount.wants/sysroot-writable.mount
25@@ -0,0 +1 @@
26+../sysroot-writable.mount
27\ No newline at end of file
28diff --git "a/factory/usr/lib/systemd/system/run-mnt-ubuntu\\x2ddata.mount.wants/sysroot-writable.mount" "b/factory/usr/lib/systemd/system/run-mnt-ubuntu\\x2ddata.mount.wants/sysroot-writable.mount"
29deleted file mode 120000
30index 0d3b832..0000000
31--- "a/factory/usr/lib/systemd/system/run-mnt-ubuntu\\x2ddata.mount.wants/sysroot-writable.mount"
32+++ /dev/null
33@@ -1 +0,0 @@
34-../sysroot-writable.mount
35\ No newline at end of file
36diff --git a/factory/usr/lib/systemd/system/sysroot-writable.mount b/factory/usr/lib/systemd/system/sysroot-writable.mount
37index 8d614a5..1006ff7 100644
38--- a/factory/usr/lib/systemd/system/sysroot-writable.mount
39+++ b/factory/usr/lib/systemd/system/sysroot-writable.mount
40@@ -1,15 +1,15 @@
41 [Unit]
42 Before=initrd-cleanup.target
43-After=run-mnt-ubuntu\x2ddata.mount
44+After=run-mnt-data.mount
45 After=run-mnt-base.mount
46 After=sysroot.mount
47 After=the-tool.service
48-Requires=run-mnt-ubuntu\x2ddata.mount
49+Requires=run-mnt-data.mount
50 Requires=run-mnt-base.mount
51 Requires=sysroot.mount
52 Requires=the-tool.service
53 Wants=populate-writable.service
54 [Mount]
55-What=/run/mnt/ubuntu-data
56+What=/run/mnt/data
57 Type=none
58 Options=bind

Subscribers

People subscribed via source and target branches