[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: (ITS#8707) slapd: Add systemd service notification support
On Thu, 10 Aug 2017 02:16:56 +0900,
Ryan Tandy wrote:
> > I've revised the patch for master to check if the listener initialization
> > suceeds or not. With this patch, `slapd -d0` can return non-zero exit code
> > if the listener initialization fails, thus systemd can detect slapd.service
> > startup failure.
>
> Nice catch! That change looks correct to me, but I hope Howard will check it too. Actually it's probably relevant for any init system; possibly worth splitting out as its own commit?
This patch does NOT affect SysV-init system because an init script
for slapd normally does NOT use the -d option.
> Just checking, do you plan to add the service file and its installation (conditional on enabled systemd support) in this branch as well, or should I work on that myself if I want it?
Yes, I'll do that later.
Thank you.
--
-- Name: SATOH Fumiyasu @ OSS Technology Corp. (fumiyas @ osstech co jp)
-- Business Home: http://www.OSSTech.co.jp/
-- GitHub Home: https://GitHub.com/fumiyas/
-- PGP Fingerprint: BBE1 A1C9 525A 292E 6729 CDEC ADC2 9DCA 5E1C CBCA