cmd/snap-bootstrap: mount base directly on /sysroot #14537
Conversation
Mount the base directly on /sysroot in UC's run mode, while for hybrid we mount /run/mnt/data instead of the /run/mnt/sysroot symlink that was pointing to the former.
alfonsosanchezbeato
added
the
Run nested
|
Opening as draft as we will need canonical/core-initrd#265 landed first. As a side note, we should discuss how to handle this initramfs/snap-bootstrap interlocks in an easier way. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #14537 +/- ##
========================================
Coverage 78.85% 78.86%
========================================
Files 1079 1080 +1
Lines 145615 145813 +198
========================================
+ Hits 114828 114997 +169
- Misses 23601 23619 +18
- Partials 7186 7197 +11
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
| } | ||
| } | ||
|
|
||
| // Do a daemon reload so systemd knows about the new mount units |
There was a problem hiding this comment.
will it also perform the mount then?
There was a problem hiding this comment.
Not really, as the new mount unit has an After=snap-initramfs-mounts.service dependency
Mount the base directly on /sysroot in all UC modes, while for hybrid
we mount /run/mnt/data instead of the /run/mnt/sysroot symlink that
was pointing to the former. With this change we do not need to have
the /run/mnt/base mount anymore.