WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] add LSB header to xen-watchdog runlevel script [and 1 more messages]
From: Olaf Hering <olaf@xxxxxxxxx>
Date: Mon, 26 Jul 2010 13:05:53 +0200
Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, Tim Deegan <Tim.Deegan@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Mon, 26 Jul 2010 04:21:35 -0700
Dkim-signature: v=1; a=rsa-sha1; c=relaxed/relaxed; t=1280142359; l=875; s=domk; d=aepfle.de; h=In-Reply-To:Content-Type:MIME-Version:References:Subject:Cc:To:From: Date:X-RZG-CLASS-ID:X-RZG-AUTH; bh=zE3ohq6lzV/K2oCxKJwJXQ9StPY=; b=tVb0BjYL+KPKdZzDNpUDOCJr9mpcn+hcQslXn228bGuUBHmOg84YqCZzsp06alt3ZyP baVkE/qPKsVrnk6aZRSHtpz6t6JmV13NyFoxeTn4L0bbYEpUF+c57j5yTvysyD1FdfcaX ek9cgLdQAhdsmKfmHRMLIb4jmzwTTc4uKoI=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <19533.25085.23971.650621@xxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20100723121324.GB3540@xxxxxxxxx> <19529.51008.134045.602693@xxxxxxxxxxxxxxxxxxxxxxxx> <20100726092344.GS13291@xxxxxxxxxxxxxxxxxxxxxxx> <20100723173620.GA7603@xxxxxxxxx> <19529.55727.859057.248861@xxxxxxxxxxxxxxxxxxxxxxxx> <20100723184317.GA9102@xxxxxxxxx> <19533.24220.728908.880255@xxxxxxxxxxxxxxxxxxxxxxxx> <1280139362.5872.6932.camel@xxxxxxxxxxxxxxxxxxxxxx> <19533.25085.23971.650621@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.20 (2009-06-14)
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