URL: https://github.com/SSSD/sssd/pull/143 Author: fidencio Title: #143: Explicitly add ordering dependency for the responders' sockets Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/143/head:pr143 git checkout pr143
From 8da21df22becaab7e3ce67a4feda252a875ea709 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabiano=20Fid=C3=AAncio?= <[email protected]> Date: Sat, 4 Feb 2017 18:12:22 +0100 Subject: [PATCH] SYSTEMD: Add "After=sssd.service" to the responders' sockets units MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit While debugging the whole breakage reported by Stric I've noticed that the NSS socket has been starting up the NSS responder _before_ SSSD being up. Leading us to a chaotic situation. By adding this ordering explicitly we can avoid the reported situation. Interesting that I haven't seen the same behaviour when starting/stopping the socket after the system is up. I also haven't noticed any kind of problem caused by explicitly adding "After=sssd.service" to the unit files. Resolves: https://fedorahosted.org/sssd/ticket/3298 Signed-off-by: Fabiano Fidêncio <[email protected]> --- src/sysv/systemd/sssd-autofs.socket.in | 1 + src/sysv/systemd/sssd-nss.socket.in | 1 + src/sysv/systemd/sssd-pac.socket.in | 1 + src/sysv/systemd/sssd-pam-priv.socket.in | 1 + src/sysv/systemd/sssd-pam.socket.in | 1 + src/sysv/systemd/sssd-ssh.socket.in | 1 + src/sysv/systemd/sssd-sudo.socket.in | 1 + 7 files changed, 7 insertions(+) diff --git a/src/sysv/systemd/sssd-autofs.socket.in b/src/sysv/systemd/sssd-autofs.socket.in index 8e0e882..e8451ff 100644 --- a/src/sysv/systemd/sssd-autofs.socket.in +++ b/src/sysv/systemd/sssd-autofs.socket.in @@ -2,6 +2,7 @@ Description=SSSD AutoFS Service responder socket Documentation=man:sssd.conf(5) BindsTo=sssd.service +After=sssd.service [Socket] ListenStream=@pipepath@/autofs diff --git a/src/sysv/systemd/sssd-nss.socket.in b/src/sysv/systemd/sssd-nss.socket.in index 530fa0c..6965015 100644 --- a/src/sysv/systemd/sssd-nss.socket.in +++ b/src/sysv/systemd/sssd-nss.socket.in @@ -2,6 +2,7 @@ Description=SSSD NSS Service responder socket Documentation=man:sssd.conf(5) BindsTo=sssd.service +After=sssd.service [Socket] ListenStream=@pipepath@/nss diff --git a/src/sysv/systemd/sssd-pac.socket.in b/src/sysv/systemd/sssd-pac.socket.in index cb1bd68..df8c8c6 100644 --- a/src/sysv/systemd/sssd-pac.socket.in +++ b/src/sysv/systemd/sssd-pac.socket.in @@ -2,6 +2,7 @@ Description=SSSD PAC Service responder socket Documentation=man:sssd.conf(5) BindsTo=sssd.service +After=sssd.service [Socket] ListenStream=@pipepath@/pac diff --git a/src/sysv/systemd/sssd-pam-priv.socket.in b/src/sysv/systemd/sssd-pam-priv.socket.in index 84b8caa..c2ddca9 100644 --- a/src/sysv/systemd/sssd-pam-priv.socket.in +++ b/src/sysv/systemd/sssd-pam-priv.socket.in @@ -3,6 +3,7 @@ Description=SSSD PAM Service responder private socket Documentation=man:sssd.conf(5) BindsTo=sssd.service BindsTo=sssd-pam.socket +After=sssd.service [Socket] Service=sssd-pam.service diff --git a/src/sysv/systemd/sssd-pam.socket.in b/src/sysv/systemd/sssd-pam.socket.in index 9554785..ad88be7 100644 --- a/src/sysv/systemd/sssd-pam.socket.in +++ b/src/sysv/systemd/sssd-pam.socket.in @@ -3,6 +3,7 @@ Description=SSSD PAM Service responder socket Documentation=man:sssd.conf(5) BindsTo=sssd.service BindsTo=sssd-pam-priv.socket +After=sssd.service [Socket] ListenStream=@pipepath@/pam diff --git a/src/sysv/systemd/sssd-ssh.socket.in b/src/sysv/systemd/sssd-ssh.socket.in index b13c87c..8f31d52 100644 --- a/src/sysv/systemd/sssd-ssh.socket.in +++ b/src/sysv/systemd/sssd-ssh.socket.in @@ -2,6 +2,7 @@ Description=SSSD SSH Service responder socket Documentation=man:sssd.conf(5) BindsTo=sssd.service +After=sssd.service [Socket] ListenStream=@pipepath@/ssh diff --git a/src/sysv/systemd/sssd-sudo.socket.in b/src/sysv/systemd/sssd-sudo.socket.in index 0b6c0d9..d7b0c87 100644 --- a/src/sysv/systemd/sssd-sudo.socket.in +++ b/src/sysv/systemd/sssd-sudo.socket.in @@ -2,6 +2,7 @@ Description=SSSD Sudo Service responder socket Documentation=man:sssd.conf(5) BindsTo=sssd.service +After=sssd.service [Socket] ListenStream=@pipepath@/sudo
_______________________________________________ sssd-devel mailing list -- [email protected] To unsubscribe send an email to [email protected]
