AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20080610170434.3a6e2852@ripper.onstor.net>
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<chris.vandever@onstor.com>,<sripal.surendiran@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	0	BB375AF679D4A34E9CA8DFA650E2B04E03E9A8DE@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Tue, 10 Jun 2008 17:05:01 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Chris Vandever" <chris.vandever@onstor.com>
Cc: Sripal Surendiran (HCL) <sripal.surendiran@onstor.com>
Subject: Re: Reboot code: take a look...
Message-ID: <20080610170501.3c31ff8d@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E03E9A8DE@onstor-exch02.onstor.net>
References: <BB375AF679D4A34E9CA8DFA650E2B04E03E9A8DE@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

OK, well, I was hoping to jump on this today, but now with the
surprise! all hands meeting tomorrow, I won't be able to get on it
until tomorrow or Thursday I suspect.

On Mon, 9 Jun 2008 22:37:57 -0700 "Chris Vandever"
<chris.vandever@onstor.com> wrote:

> I'm thinking of something along the following lines.  This is VERY
> preliminary.  I just wanted to give you an idea of what I think we
> need. The guts are in cmd-utils.c.  I removed the clustering
> equivalent from cluster-util.c, and updated the callers in
> cluster-contrl-cfg.c and cluster-server-rpc.c.  The other files are
> just minor cleanup.  The new utils_rebootSystem() function works fine
> for what I'm doing, although I should be calling it with more flags
> set.  I'm executing very little of it.
> 
> The problem is I'm clueless when it comes to exactly what should be
> done and in what order, not to mention on what platforms.  Feel free
> to play with it or start over from scratch.  If nothing else, it
> gives you an idea of the type of interface I'd like to see.  I
> believe the nodeDownReason and appName should be required.  If
> clustering and vsd end up being the only callers then they probably
> don't need the flags argument.  (Also, I'm sure the flags make no
> sense, but hopefully it gives you an idea of what might be needed.)
> 
> I was hoping we could use the routine in other code as well, like
> chassisd, pm, nfxsh, and sct taskmgr.
> 
> The code is in ~chrisv/p4/dev3/060908Backup, and in my p4 client,
> chrisv-dev3.
> 
> ChrisV
> ----------
> 
> //depot/dev/nfx-tree/code/sm-gns/gns-api.c#6 - edit change 29608
> (xtext) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/sm-utils/cmd-utils.c#24 - edit change 29608
> (xtext) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/sm-utils/cmd-utils.h#5 - edit change 29608
> (xtext) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/ssc-cluster/cluster-api.h#3 - edit change
> 29608 (text) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/ssc-cluster/cluster-contrl-cfg.c#28 - edit
> change 29608 (text) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/ssc-cluster/cluster-gns-api.c#9 - edit
> change 29608 (xtext) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/ssc-cluster/cluster-server-rpc.c#5 - edit
> change 29608 (text) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/ssc-cluster/cluster-util.c#15 - edit change
> 29608 (text) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/ssc-cluster/cluster-util.h#11 - edit change
> 29608 (text) by chrisv@chrisv-dev3
> //depot/dev/nfx-tree/code/ssc-cluster/cluster-vsvr-api.c#10 - edit
> change 29608 (text) by chrisv@chrisv-dev3
> 
> Prerequisite for #23399 (clustering needs to restart apps when it
> gets a new clusDb): Create a utility function apps can call to
> initiate a reboot, with all of the appropriate notifications
> regarding the reason for the reboot.  Eliminate the function in
> clustering that does the same thing, incorrectly.
> 
