[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#8707) slapd: Add systemd service notification support



Hi,

I've added slapd.service file. Please check.

* https://github.com/osstech-jp/openldap/pull/4
* https://patch-diff.githubusercontent.com/raw/osstech-jp/openldap/pull/4.patch

On Thu, 10 Aug 2017 19:26:48 +0900,
fumiyas 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.

-- 
-- 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