diff options
| author | Franck Bui <fbui@suse.com> | 2020-06-11 09:41:14 +0200 |
|---|---|---|
| committer | Franck Bui <fbui@suse.com> | 2020-07-01 16:34:42 +0200 |
| commit | 045c5faf5a3ef51cd39f738694e06c11a73f54b7 (patch) | |
| tree | dfdaafcb042119a4770c5547f0a2f8f140ed11bb /src/fstab-generator | |
| parent | doc: add recentry introduced transient settings (diff) | |
| download | systemd-045c5faf5a3ef51cd39f738694e06c11a73f54b7.tar.xz systemd-045c5faf5a3ef51cd39f738694e06c11a73f54b7.zip | |
fstab-generator: extra dependencies specified in fstab should be applied to the mount unit
If an entry in fstab uses "x-systemd.automount" option and also asks for
additionnal dependencies via x-systemd.requires or such, then the dependencies
were applied to the automount unit.
But this unlikely to do the right thing and is inconsistent with what's done
for network mounts.
Indeed when an fstab entries has "_netdev,x-systemd.automount" options, the
dependencies against the network requested by "_netdev" are (correctly) applied
to the mount unit only and the automount unit remains ordered against
local-fs.target.
The same logic should be followed when extra deps are specified via the mount
options as automount units should always be ordered against local-fs.target.
Note: in general explicit deps specified via mount options should be used with
care and should be used to specify dependencies on other mount units only as it
can easily create ordering cycles otherwise like it's been seen in
https://github.com/systemd/systemd-stable/issues/69. Mount units (as well as
automount ones) are ordered before local-fs.target by default which is a
low-level target that most other units depend on.
Diffstat (limited to 'src/fstab-generator')
| -rw-r--r-- | src/fstab-generator/fstab-generator.c | 17 |
1 files changed, 1 insertions, 16 deletions
diff --git a/src/fstab-generator/fstab-generator.c b/src/fstab-generator/fstab-generator.c index 77b90e1a9b..39e91a9c6b 100644 --- a/src/fstab-generator/fstab-generator.c +++ b/src/fstab-generator/fstab-generator.c @@ -406,7 +406,7 @@ static int add_mount( SET_FLAG(flags, NOFAIL, true); } - if (!(flags & AUTOMOUNT) && opts) { + if (opts) { r = write_after(f, opts); if (r < 0) return r; @@ -530,21 +530,6 @@ static int add_mount( "Documentation=man:fstab(5) man:systemd-fstab-generator(8)\n", source); - if (opts) { - r = write_after(f, opts); - if (r < 0) - return r; - r = write_requires_after(f, opts); - if (r < 0) - return r; - r = write_before(f, opts); - if (r < 0) - return r; - r = write_requires_mounts_for(f, opts); - if (r < 0) - return r; - } - fprintf(f, "\n" "[Automount]\n" |