Re: [Xen-devel] [PATCH] add LSB header to xen-watchdog runlevel script [and 1 more messages]

[Olaf Hering <olaf@xxxxxxxxx>]

On Mon, Jul 26, Ian Jackson wrote:

> Ian Campbell writes ("Re: [Xen-devel] [PATCH] add LSB header to xen-watchdog 
> runlevel script [and 1 more messages]"):
> > On Mon, 2010-07-26 at 11:08 +0100, Ian Jackson wrote:
> > > I meant: adding these lines would seem to be likely to cause some
> > > systems to automatically start the watchdog when previously they would
> > > not have done so.  I wanted to know whether that was a good idea.
> > 
> > IMHO no -- enabling a watchdog should be an explicit admin action.
> 
> So I conclude that you think the patch is wrong ?

Ian,

the patch is not wrong. If it were, the chkconfig line had to be removed
as well. Both lines just tell the tools about the ordering in the
specifieed rulevels, when the admin actually wants to have the script
enabled by calling 'chkconfig -a xen-watchdog' or 'insserv watchdog'.

Olaf


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel