Mailing List Archive

[PATCH] Correction at the time of the slapd start delay
Hi Dejan,

It becomes the error at the time of a start delay of ldapserver now.
Therefore I send a patch revising it.

I did pull request to github.
https://github.com/nozawat/resource-agents/commit/bbc0934ea7b906f818e3c7e00bfaaa61ae46bc51

Regards,
Tomo

--
Tomoya Nozawa

_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
Re: [PATCH] Correction at the time of the slapd start delay [ In reply to ]
Hi Tomoya-san,

On Wed, May 09, 2012 at 06:14:34PM +0900, nozawat@gmail.com wrote:
> Hi Dejan,
>
> It becomes the error at the time of a start delay of ldapserver now.
> Therefore I send a patch revising it.
>
> I did pull request to github.
> https://github.com/nozawat/resource-agents/commit/bbc0934ea7b906f818e3c7e00bfaaa61ae46bc51

OK. Though the description doesn't seem to fit the change.
Shouldn't it be something like this:

Medium: slapd: always set the exit code correctly in monitor

In case ldapsearch would exit with an error code different from
49, the exit code may have been wrong.

Cheers,

Dejan

> Regards,
> Tomo
>
> --
> Tomoya Nozawa
>
> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
Re: [PATCH] Correction at the time of the slapd start delay [ In reply to ]
Hi again,

On Wed, May 09, 2012 at 06:14:34PM +0900, nozawat@gmail.com wrote:
> Hi Dejan,

Since recently it seems as if all emails to linux-ha-dev are
addressed to me. Happily, I'm not the only one around watching.
Nor should I feel as the only responsible for the state of
affairs in our project. I just thought I should straighten things
before it gets too late ;-)

Cheers,

Dejan

P.S. Not to be taken personally!
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
Re: [PATCH] Correction at the time of the slapd start delay [ In reply to ]
Hi Dejan,

Thank you for comment.
I made modifications and did pull request again.

Regards,
Tomo

On Wed, 9 May 2012 11:47:50 +0200
Dejan Muhamedagic <dejan@suse.de> wrote:

> Hi Tomoya-san,
>
> On Wed, May 09, 2012 at 06:14:34PM +0900, nozawat@gmail.com wrote:
> > Hi Dejan,
> >
> > It becomes the error at the time of a start delay of ldapserver now.
> > Therefore I send a patch revising it.
> >
> > I did pull request to github.
> > https://github.com/nozawat/resource-agents/commit/bbc0934ea7b906f818e3c7e00bfaaa61ae46bc51
>
> OK. Though the description doesn't seem to fit the change.
> Shouldn't it be something like this:
>
> Medium: slapd: always set the exit code correctly in monitor
>
> In case ldapsearch would exit with an error code different from
> 49, the exit code may have been wrong.
>
> Cheers,
>
> Dejan
>
> > Regards,
> > Tomo
> >
> > --
> > Tomoya Nozawa
> >
> > _______________________________________________________
> > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> > Home Page: http://linux-ha.org/
> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/

--
Tomoya Nozawa

_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
Re: [PATCH] Correction at the time of the slapd start delay [ In reply to ]
Hi,

On Thu, May 10, 2012 at 01:49:45PM +0900, 野沢 智也 wrote:
> Hi Dejan,
>
> Thank you for comment.
> I made modifications and did pull request again.

Thanks for the effort! I just pulled the patch.

Cheers,

Dejan

> Regards,
> Tomo
>
> On Wed, 9 May 2012 11:47:50 +0200
> Dejan Muhamedagic <dejan@suse.de> wrote:
>
> > Hi Tomoya-san,
> >
> > On Wed, May 09, 2012 at 06:14:34PM +0900, nozawat@gmail.com wrote:
> > > Hi Dejan,
> > >
> > > It becomes the error at the time of a start delay of ldapserver now.
> > > Therefore I send a patch revising it.
> > >
> > > I did pull request to github.
> > > https://github.com/nozawat/resource-agents/commit/bbc0934ea7b906f818e3c7e00bfaaa61ae46bc51
> >
> > OK. Though the description doesn't seem to fit the change.
> > Shouldn't it be something like this:
> >
> > Medium: slapd: always set the exit code correctly in monitor
> >
> > In case ldapsearch would exit with an error code different from
> > 49, the exit code may have been wrong.
> >
> > Cheers,
> >
> > Dejan
> >
> > > Regards,
> > > Tomo
> > >
> > > --
> > > Tomoya Nozawa
> > >
> > > _______________________________________________________
> > > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> > > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> > > Home Page: http://linux-ha.org/
> > _______________________________________________________
> > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> > Home Page: http://linux-ha.org/
>
> --
> Tomoya Nozawa
>
> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/