All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clvmd init: populate Required-Start/Stop at configure time
@ 2010-08-11  7:24 Fabio M. Di Nitto
  2010-08-11  7:33 ` Fabio M. Di Nitto
  0 siblings, 1 reply; 2+ messages in thread
From: Fabio M. Di Nitto @ 2010-08-11  7:24 UTC (permalink / raw)
  To: lvm-devel

Based on auto-detection or user requested cluster managers for clvmd,
set appropriate Required-Start and Required-Stop at configure time.

Reorder the checks for user selected cluster managers to match auto
detected ones, to be consistent in the output.

Add special case for qdiskd that?s started after cman/lock_gulmd for
RHEL-4/RHEL-5.

Tested on RHEL-4, 5 and 6 and it seems to work fine. I?d really
appreciate some extra testings tho.

Fabio
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: clvmd_init_proper_required_start_stop.diff
URL: <http://listman.redhat.com/archives/lvm-devel/attachments/20100811/b883a708/attachment.ksh>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [PATCH] clvmd init: populate Required-Start/Stop at configure time
  2010-08-11  7:24 [PATCH] clvmd init: populate Required-Start/Stop at configure time Fabio M. Di Nitto
@ 2010-08-11  7:33 ` Fabio M. Di Nitto
  0 siblings, 0 replies; 2+ messages in thread
From: Fabio M. Di Nitto @ 2010-08-11  7:33 UTC (permalink / raw)
  To: lvm-devel

On 8/11/2010 9:24 AM, Fabio M. Di Nitto wrote:
> Based on auto-detection or user requested cluster managers for clvmd,
> set appropriate Required-Start and Required-Stop at configure time.
> 
> Reorder the checks for user selected cluster managers to match auto
> detected ones, to be consistent in the output.
> 
> Add special case for qdiskd that?s started after cman/lock_gulmd for
> RHEL-4/RHEL-5.
> 
> Tested on RHEL-4, 5 and 6 and it seems to work fine. I?d really
> appreciate some extra testings tho.
> 
> Fabio


Forgot one side note, my autoreconf diff is somewhat gigantic and I
don?t feel comfortable to push it. I can commit the patch, but i?d
prefer if somebody else would run autoreconf for me.

Fabio



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-08-11  7:33 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-11  7:24 [PATCH] clvmd init: populate Required-Start/Stop at configure time Fabio M. Di Nitto
2010-08-11  7:33 ` Fabio M. Di Nitto

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.