URL: https://github.com/SSSD/sssd/pull/143 Title: #143: Explicitly add ordering dependency for the responders' sockets
jhrozek commented: """ On Tue, Feb 28, 2017 at 07:51:26AM -0800, fidencio wrote: > On Tue, Feb 28, 2017 at 4:25 PM, lslebodn <[email protected]> wrote: > > > On (28/02/17 04:27), Jakub Hrozek wrote: > > >I tested the patches both during the VM operation bringing different > > responders up and during system boot. I haven't see any issues. > > > > > > > A tiny question. > > Do we need to add After to all sockets? > > The ideal would be that socket will trigger starting of responder > > and monitor will be started as a dependency of responder. > > > > I thought it is required just for nss due to some circular dependency. > > > > I would appreciate explanation. So if someone decide to remove it in future > > he will consider problematic use-case. > > > > Lukáš, > > That was a recommendation of Lukáš Nykrýn that BindsTo and After must go > together. It's mentioned in the commit message. > Lukáš Nykrýn also mentioned they will fix their docs about this. I think Lukas was asking about something else -- would it be enough to only add those changes to the nss socket file? """ See the full comment at https://github.com/SSSD/sssd/pull/143#issuecomment-283155833
_______________________________________________ sssd-devel mailing list -- [email protected] To unsubscribe send an email to [email protected]
