From owner-freebsd-hackers@FreeBSD.ORG Mon May 1 21:26:42 2006 Return-Path: X-Original-To: hackers@freebsd.org Delivered-To: freebsd-hackers@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8ED7916A400 for ; Mon, 1 May 2006 21:26:42 +0000 (UTC) (envelope-from cokane@mail.cokane.org) Received: from smtp1.fuse.net (mail-out1.fuse.net [216.68.8.174]) by mx1.FreeBSD.org (Postfix) with ESMTP id 1401D43D45 for ; Mon, 1 May 2006 21:26:41 +0000 (GMT) (envelope-from cokane@mail.cokane.org) Received: from gx6.fuse.net ([72.49.162.239]) by smtp1.fuse.net (InterMail vM.6.01.04.04 201-2131-118-104-20050224) with ESMTP id <20060501212641.KJCQ3018.smtp1.fuse.net@gx6.fuse.net> for ; Mon, 1 May 2006 17:26:41 -0400 Received: from mail.cokane.org ([72.49.162.239]) by gx6.fuse.net (InterMail vG.1.02.00.02 201-2136-104-102-20041210) with ESMTP id <20060501212641.MEPB4722.gx6.fuse.net@mail.cokane.org> for ; Mon, 1 May 2006 17:26:41 -0400 Received: (qmail 2270 invoked by uid 1000); 1 May 2006 17:28:01 -0400 Date: Mon, 1 May 2006 17:28:01 -0400 From: Coleman Kane To: Brooks Davis Message-ID: <20060501212801.GA2254@pint.candc.home> References: <20060424131508.GB23163@pint.candc.home> <444CD48A.4060501@centtech.com> <444CE475.30104@centtech.com> <20060430231621.GA551@pint.candc.home> <44557F34.3020906@centtech.com> <20060501190645.GB4315@odin.ac.hmc.edu> <44565DD2.1020604@centtech.com> <20060501191447.GD4315@odin.ac.hmc.edu> <44565E74.3060801@centtech.com> <20060501192920.GE4315@odin.ac.hmc.edu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060501192920.GE4315@odin.ac.hmc.edu> User-Agent: Mutt/1.5.11 X-Mailman-Approved-At: Mon, 01 May 2006 21:33:22 +0000 Cc: hackers@freebsd.org Subject: Re: [PATCH] Fancy rc startup style RFC X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 May 2006 21:26:42 -0000 On Mon, May 01, 2006 at 12:29:20PM -0700, Brooks Davis wrote: > On Mon, May 01, 2006 at 02:16:04PM -0500, Eric Anderson wrote: > > Brooks Davis wrote: > > >On Mon, May 01, 2006 at 02:13:22PM -0500, Eric Anderson wrote: > > >>Brooks Davis wrote: > > >>>On Sun, Apr 30, 2006 at 10:23:32PM -0500, Eric Anderson wrote: > > >>>>Coleman Kane wrote: > > >>>>>On Mon, Apr 24, 2006 at 09:45:09AM -0500, Eric Anderson wrote: > > >>>>>>Eric Anderson wrote: > > >>>>>> > > >>>>>>Actually, some other things got changed somewhere in the history, > > >>>>>>that broke some things and assumptions I was making. This patch has > > >>>>>>them fixed, and I've tested it with all the different options: > > >>>>>> > > >>>>>>http://www.googlebit.com/freebsd/patches/rc_fancy.patch-9 > > >>>>>> > > >>>>>>It's missing the defaults/rc.conf diffs, but you should already know > > >>>>>>those. > > >>>>>> > > >>>>>> > > >>>>>>Eric > > >>>>>> > > >>>>>I have a new patch (to 7-CURRENT) of the "fancy_rc" updates. > > >>>>> > > >>>>>This allows the use of: > > >>>>>rc_fancy="YES" ---> Turns on fancy reporting (w/o color) > > >>>>>rc_fancy_color="YES" ---> Turns on fancy reporting (w/ color), needs > > >>>>> rc_fancy="YES" > > >>>>>rc_fancy_colour="YES" ---> Same as above for you on the other side of > > >>>>> the pond. > > >>>>>rc_fancy_verbose="YES" --> Turn on more verbose activity messages. > > >>>>> This will cause what appear to be "false > > >>>>> positives", where an unused service is > > >>>>> "OK" instead of "SKIP". > > >>>>> > > >>>>>You can also customize the colors, the widths of the message > > >>>>>brackets (e.g. [ OK ] vs. [ OK ]), the screen width, and > > >>>>>the contents of the message (OK versus GOOD versus BUENO). > > >>>>> > > >>>>>Also, we have the following message combinations: > > >>>>>OK ---> Universal good message > > >>>>>SKIP,SKIPPED ---> Two methods for conveying the same idea? > > >>>>>ERROR,FAILED ---> Ditto above, for failure cases > > >>>>> > > >>>>>Should we just have 3 different messages, rather than 5 messages > > >>>>>in 3 categories? > > >>>>Yes, that's something that started with my first patch, and never got > > >>>>ironed out. I think it should be: > > >>>>OK > > >>>>SKIPPED > > >>>>FAILED > > >>>>and possibly also: > > >>>>ERROR > > >>>> > > >>>>The difference between FAILED and ERROR would be that FAILED means the > > >>>>service did not start at all, and ERROR means it started but had some > > >>>>kind of error response. > > >>>FAILED vs ERROR seems confusing. I'd be inclined toward WARNING vs > > >>>FAILED or ERROR. > > >>True, however I still see a difference between FAILED and WARNING. For > > >>instance, as an example: a FAILED RAID is different than a RAID with a > > >>WARNING. > > > > > >For that level of detail, the ability to provide additional output seems > > >like the appropriate solution. > > > > Yes, true, but you'd still want to show something (I would think) in the > > [ ]'s to keep it consistent. > > My feeling is that anything short of complete success should report > WARNING and a message unless it actually totally failed in which case > FAILED or ERROR (I slightly perfer ERROR) should be used. > > -- Brooks What situations are we determining get flagged as ERROR versus FAILED? Is FAILED considered to be 'I was able to run the command, but it returned an error code', versus ERROR being 'I could not even run the command!' like bad path, file not found, etc... This point still kind of confuses me (and needs to be well defined). I am an advocate of having three distinct messages: OK, SKIPPED, ERROR. And not even bothering with the different types of ERROR/FAILED other than having extra reporting output. Of course, I am still willing to implement what the consensus is. -- Coleman Kane