On (26/08/16 13:16), Victor Tapia wrote:
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.
I ran tests and I do not understand why but some tests on rhel6
were stucked in weird state. I'm not sure whether it's caused by your patch
or by wrong expectation in tests. I will also try to test with 1.13.
But so far it seems to work with systemd.
LS