Hi,
Sorry for the late response, I've been out for nearly two weeks. I'm
attaching a new version of the patch to review.
Does upstart/init script need to be updated as well?
We have some generic in src/sysv/sssd.in
Upstart does need to keep track of the pidfile creation. I can't see a
service file, so I'll prepare one.
Regarding sysv, due to its sequential nature, it's dangerous to include
an infinite loop because it could stuck the normal boot of the machine.
I included a timeout in the loop to avoid this, please let me know what
you think about it.
BTW the patch causes some failures for me on rhel6.
According to logs, the file /var/run/sssd.pid
did not exist in some cases.
Please also remove changes in src/sysv/systemd/sssd.service.in
in next version. For systemd only I would prefer "Type=notify".
Done, I've switched to Type=notify for systemd.
Thanks,
Victor