[PATCH] Add systemd-user-sessions dependency to systemd service file.

[Ian Lee]

Please see attached patch. Fixes issues when shutting down/restarting.

[Jakub Hrozek]

On Wed, Aug 06, 2014 at 03:18:04PM +0100, Ian Lee wrote:
>  Please see attached patch. Fixes issues when shutting
> down/restarting. 
Thanks for the patch!

Looks fine, but I was thinking lately if we should start even sooner to
allow SSSD users to be listed in cron jobs. I was considering being as
agressive as starting after basic.target and relying on our network
detection (and not wait for network-online for instance)

Any other opinions?

[Jakub Hrozek]

On Wed, Aug 06, 2014 at 06:33:44PM +0200, Jakub Hrozek wrote:
>  On Wed, Aug 06, 2014 at 03:18:04PM +0100, Ian Lee wrote:
>  > Please see attached patch. Fixes issues when shutting down/restarting.
>  
>  Thanks for the patch!
>  
>  Looks fine, but I was thinking lately if we should start even sooner to
>  allow SSSD users to be listed in cron jobs. I was considering being as
>  agressive as starting after basic.target and relying on our network
>  detection (and not wait for network-online for instance)
>  
>  Any other opinions? 
Hi,

I pinged the systemd developers and in a short IRC chat they suggested
also adding nss-user-lookup.target.

Because SSSD is a NSS user provider, we should add:
    Before=nss-user-lookup.target
    Wants=nss-user-lookup.target

For more details, see man systemd.special, section "Special passive
system units". Also the diagram in "man 7 bootup" is helpful.

Ian, do you agree? Shall I fixup your patch or would you like to send a
new version yourself?