AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<rendell.fong@onstor.com>,<jonathan.goldick@onstor.com>,<brian.stark@onstor.com>
MAID:1
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#imap/andys@onstor.net@onstor-exch02.onstor.net/INBOX	48910	BB375AF679D4A34E9CA8DFA650E2B04E09624C02@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Wed, 6 Aug 2008 17:56:33 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Rendell Fong" <rendell.fong@onstor.com>
Cc: "Jonathan Goldick" <jonathan.goldick@onstor.com>, "Brian Stark"
 <brian.stark@onstor.com>
Subject: Re: prelim review of Node Reset FS
Message-ID: <20080806175633.63639053@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E09624C02@onstor-exch02.onstor.net>
References: <BB375AF679D4A34E9CA8DFA650E2B04E0B31A44D@onstor-exch02.onstor.net>
 <BB375AF679D4A34E9CA8DFA650E2B04E09624C02@onstor-exch02.onstor.net>
Organization: Onstor
X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.8.20; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit


I did not have any extra comments beyond Jon's, so you're good to go
from me.

On Wed, 6 Aug 2008 14:41:26 -0700 "Rendell Fong"
<rendell.fong@onstor.com> wrote:

> I've made updates based on your second set of comments.  
> 
> I'm using Citrix so I think the pagination is different for some
> reason. It shows that there are only 13 pages on the doc so the page
> numbers you've referenced don't match up.
> 
> After Brian completes his testing, I'll make the final updates and
> send it out to the general audience.
> 
> Thanks,
> Rendell
> 
> _____________________________________________
> From: Jonathan Goldick 
> Sent: Wednesday, August 06, 2008 1:56 PM
> To: Rendell Fong; Andy Sharp; Brian Stark
> Subject: RE: prelim review of Node Reset FS
> 
> 1.	Page 4, rebuild the table of contents
> 2.	Page 14, item #4 try halt on an already halted system
> 3.	Page 14, Try halt on the board you have logged into when the
> other node is already halted.
> 
> The rest looks good.
> 
> _____________________________________________
> From: Rendell Fong 
> Sent: Wednesday, August 06, 2008 1:04 PM
> To: Jonathan Goldick; Andy Sharp; Brian Stark
> Subject: RE: prelim review of Node Reset FS
> 
> See inline comments.  Revised file is enclosed.
> 
> _____________________________________________
> From: Jonathan Goldick 
> Sent: Wednesday, August 06, 2008 11:55 AM
> To: Rendell Fong; Andy Sharp; Brian Stark
> Subject: RE: prelim review of Node Reset FS
> 
> 1.	Page 7, leave the numbered header in for Problem statement,
> it should be chapter 4.
> 
> 	Done.  I put the number is back. I admit I'm not a Word expert
> so don't know how it disappeared.
> 
> 2.	Page 7, the problem statement should include the fact that
> the reset button is not sufficient in lights out data centers that
> would only have the ability to power cycle the chassis when we only
> need to reset a single blade.
> 
> 	Done.
> 
> 3.	Page 9, "The node will then perform a reset/halt of itself
> as specified."  --> "The node will then perform a reset/halt of it as
> specified."
> 
> 	Done.
> 
> 4.	Page 9, You should add that an attempt to reset the node
> that you are logged into should print a warning message that "system
> reboot" or "system halt" is the preferred mechanism before prompting
> to go forward.
> 
> 	Done.  Also, removed "excluding the local node" wording from
> several statements.
> 
> 5.	Page 9, You should add that this command will not be allowed
> when the admin connected into a virtual server IP address, but only
> from an SSC IP address or the serial interface.  This restriction has
> been put in place to reduce the likelihood that the admin resets the
> wrong node when using the "-other" option.  Virtual servers can fail
> over at any time so the admin may not be on the node they think they
> are.
> 
> 	Done.
> 
> 6.	Page 10, "Once initiated, the neighboring node will remain
> halted until a "system reset" is issued, the node from which the halt
> was issued is rebooted"   This may no longer be valid after Brian's
> work in progress.
> 
> 	Yes, I know. I need his final input before this can be
> updated.
> 
> 7.	Page 11, "Node name is the local node  (Only system reboot
> will be allowed to do this)"  see note 4 above.  I think this should
> be allowed with a warning.  I don't why we might need it but I would
> not make it impossible.
>       Converted to warning instead.  User will be prompted to proceed
> or not.
> 8.	Page 14, you need to include the tests for the various
> illegal node options, and the board pull, the power cycle clearing
> things, what happens to a halted node when the other blade crashes or
> reboots.
> 
> 	I've enhance the test cases.  Hope its now sufficient. 
> 
> 
>  << File: NodeReset_v1.0.doc >> 
> _____________________________________________
> From: Rendell Fong 
> Sent: Wednesday, August 06, 2008 11:36 AM
> To: Jonathan Goldick; Andy Sharp; Brian Stark
> Subject: prelim review of Node Reset FS
> 
> Hi Guys,
> 
> Please review the enclosed func spec before I release it to the
> general audience.
> Hopefully, everything discussed in our meeting yesterday is now
> captured in this FS but I may have missed some things.
> BrianS is still doing some testing so some of the notes may need to be
> updated based on his input.
> 
> Thanks,
> Rendell 
> 
> 
>  << File: NodeReset_v1.0.doc >> 
