Hi Didier, On Tue, Nov 18, 2014 at 12:11 PM, Didier Roche <didro...@ubuntu.com> wrote: > This has 3 drawbacks: > - Duplicate symlinks for the same targets between /etc and units enabled in > /usr/lib for units which are already enabled via /usr/lib, if the admin runs > "enable"
This I think should be considered a bug in the unit file. If a unit has a /usr/lib symlink, then it is statically enabled (i.e., 'enable'/'disable' has no effect), so it should not install symlinks in /etc, and hence not have an [Install] section. At least with the current semantics. > - Wrt. the "golden image, /etc reset" approach of reducing base os > installation defaults in /etc, this is another instance of "always needs to > be there" clutter in /etc. If the package default is to start the service, > then it's better to just ship that wants.d/ symlink in the package (and thus > in /usr) instead of always having to create the symlink in /etc at package > install time or after a factory reset. I get where you are coming from, but presets will give you the same result, no? > - We are mixing sys admin information and distro default choices in the same > directories, and can't tell apart what is what. That is true. Could we perhaps improve on systemctl by printing "enabled (preset)"/"disable (preset)" for units that are in the default state? I know this does not change the fact that you have distro-default (via presets) links in /etc, but it should give the end-user a nicer experienc I guess? > We were thus thinking about having all default distro choices shipping > target symlinks in /usr/lib, and having the administrator overrides this in > /etc via systemctl. This could look similar to masking a unit, i. e. a > symlink "/etc/.../wants.d/foo -> /dev/null" overrides > "/usr/lib/.../wants.d/foo -> ../foo.service", and would be an explicit > representation of "the admin does not want foo.service to autostart" in > /etc. So you are essentially proposing to replace the preset concept? We now have: enabeld - [Install] section and symlink in /etc/**/*.wants.d/ disabled - [Install] section and no symlink in /etc/**/*.wants.d/ static - no [Install] section and symlink in /usr/lib/**/*.wants.d/ masked - symlink from the unit file-name to /dev/null in /etc you want in addition: preset (or something like that) - [Install] section and symlink in /usr/lib/**/*.wants.d/ unpreset (or something like that) - [Install] section and symlink in /usr/lib/**/*.wants.d/ and an overriding symlink in /etc pointing at /dev/null Doing 'enable' on a preset unit will then just delete the symlink to /dev/null from /etc (if it exists) and doing 'disable' will add it. This would also entail changing the current logic to check the target of /**/*.wants.d/ symlinks to see if they point to /dev/null, in which case they should be ignored. Did I understand that correctly? > However, we did notice the following: taking an unit, with an [Install] > section and a symlink to enable in via that target in /usr/lib: Yeah, I would not expect this to work with the current semantics, as it appears to be a contradiction. We can probably improve on the status-quo even if your proposal is not implemented. > - systemctl status <unit> will report "disabled", where it's actually > enabled and starting for that unit. This is a bug which should be fixed in > any case, as "disabled" is outright wrong. On IRC it was suggested that > those are "static" units, but then it should say at least that. I agree, this should be fixed to report them as 'static' (as any state in /etc apart from masking is irrelevant). > - systemctl enable <unit> will duplicate the symlink in /etc I guess this should also be dropped (though the situation here is weird as it anyway is a noop). Maybe a warning should be printed. > - If we ln -s /dev/null /etc/<…>/<…>.wants/<unit>, then systemctl status > <unit> will display the unit as being enabled, and it will activated once > the target is reached. This is also counterintuitive, as usually this means > to mask/completely disable the unit. You cannot mask an wants.d/* symlink, only the unit itself, so this is actually not defined. My understanding is that this is a concept you would like to introduce rather than a bug. I don't know how/if we should change this behaviour under the current semantics. > Part of the discussion on #systemd pointed out that the admin should then > use systemctl mask <unit>. However, that means: > - The admin can't retarget a default installed unit without recreating > another unit file Correct. But is that something we want? I mean, how I would retarget a unit file is to make a copy, edit the [Install] section and then "systemctl enable" it. I guess we should not encourage admins to go fiddling with the symlinks manually? > - There are 2 commands to "disable" an unit: mask for some, disable for > others, this can bring confusion and admins won't know the semantic > difference between the two (and indeed this is rather technical and > unintuitive) Well, the meaning we have been using so far is that statically enabled units are things that does not really make sense to disable (which is different from it being enabled by default), and for all other units the way to enable/disable them (be it by default or manually) is by installing symlinks in /etc. If the admin wants to insist to ignore the "does not make sense to disable part" and really force something to never start, then masking is the tool for that. Either way, the admin should never (need to) go poking in /etc manually, but use systemctl as the interface. > - The status reported with systemctl is still disabled when it's not. We can probably improve on that. I guess no one really explored the case of static units with [Install] sections. Even if we end up thinking of these as bugs, people can still end up with them, so we should probably make sure our tools report something sensible. > Tested with systemd 216 on fedora 21 and systemd 215 on ubuntu vivid. > > It will be great if we can come to some common grounds on how we should > separate admin choices and default distro choices, while still working on > the "remove /etc default distro configuration" . I'm happy to give a hand on > the desired solutions there. My take on this is: make sure presets are applied on "firstboot" (which I think they are), so empty /etc works just fine, and then improve on systemctl to better show the distinction between 'enabled by default' or 'enabled by choice' (and same for 'disabled'). Cheers, Tom _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel
