You are right, they can be different but the main idea was thet a user
must provide a correct node name. So I'm not sure in this patch
because it creates some kind of a loop in logic: config file depends
on a node name  from hostlist, and now you are making node name to
depend on that config file.

What I mean is: User must provide a correct node name in "hostlist"
variable. Config file is an optional parameter in case if it's name
doesn't match the node name. So I think that it's kind of overdo to
look for a right node name in the config file again.

On Wed, Mar 5, 2008 at 7:03 PM, DAIKI MATSUDA <[EMAIL PROTECTED]> wrote:
> Hi,
>
>  I have already made similar one. And I tested your 'xen0'. It has a
>  little problem. The domU's hostname is not always same as config file
>  name. So, I fixed it to pick up the target node from config file.
>
>  Regards
>  MATSUDA, Daiki
>
>  2008/2/28, Serge Dubrouski <[EMAIL PROTECTED]>:
>  > Attached.
>  >
>  >
>  >  On Thu, Feb 28, 2008 at 3:35 AM, Dejan Muhamedagic <[EMAIL PROTECTED]> 
> wrote:
>  >  > Hi Serge,
>  >  >
>  >  >
>  >  >  On Tue, Feb 26, 2008 at 09:46:14AM -0700, Serge Dubrouski wrote:
>  >  >  > Dejan -
>  >  >  >
>  >  >  > I found a compromise :-) Attached is a version of that plugin that
>  >  >  > supports following parameters:
>  >  >  >
>  >  >  > 1. hostlist. A string that has a list of controlled nodes separated 
> by
>  >  >  > space or commas. A required parameter. In a simple form its just a
>  >  >  > list of nodes. If one needs to use non-standard Xen configuration he
>  >  >  > can use extended form of this parameter: "node1_name:config1_file
>  >  >  > node2_name:config2_file". If config file isn't given it defaults to
>  >  >  > /etc/xen/node_name.cfg
>  >  >  >
>  >  >  > 2. Dom0. Name of Dom0 Xen node. A required parameter.
>  >  >  >
>  >  >  > 3. ssh_command. SSH command that is used to ssh from DomU to Dom0.
>  >  >  > Defaults to "/usr/bin/ssh -q -x -n -l root". If one wants to use SSH
>  >  >  > keys for higher security he needs to use this parameter.
>  >  >
>  >  >  This is not necessary. One can setup everything needed in
>  >  >  ~/.ssh/config on a per host basis, i.e. key or user to connect
>  >  >  with. Since the plugin always runs as root, you might leave out
>  >  >  '-l root' as well.
>  >  >
>  >  >
>  >  >  > So in this form this plugin can be configured as a clone or as a set
>  >  >  > of resources and location constraints.
>  >  >  >
>  >  >  >
>  >  >  > I'd be very pleased if this plugin gets its way into Linux-HA 
> distribution.
>  >  >  >
>  >  >
>  >  >  Sure. Could you please just drop the ssh_command parameter.
>  >  >
>  >  >  Many thanks for the contribution.
>  >  >
>  >  >  Cheers,
>  >  >
>  >  >  Dejan
>  >  >
>  >  >
>  >  >
>  >  >  >
>  >  >  > On Tue, Feb 26, 2008 at 8:45 AM, Serge Dubrouski <[EMAIL PROTECTED]> 
> wrote:
>  >  >  > >
>  >  >  > > On Mon, Feb 25, 2008 at 4:02 PM, Dejan Muhamedagic <[EMAIL 
> PROTECTED]> wrote:
>  >  >  > >  > Hi,
>  >  >  > >  >
>  >  >  > >  >
>  >  >  > >  >  On Mon, Feb 25, 2008 at 12:17:40PM -0700, Serge Dubrouski 
> wrote:
>  >  >  > >  >  > On Mon, Feb 25, 2008 at 12:10 PM, Dejan Muhamedagic <[EMAIL 
> PROTECTED]> wrote:
>  >  >  > >  >  > > Hi,
>  >  >  > >  >  > >
>  >  >  > >  >  > >
>  >  >  > >  >  > >  On Mon, Feb 25, 2008 at 11:27:38AM -0700, Serge Dubrouski 
> wrote:
>  >  >  > >  >  > >  > I would love to do that and already tried it. Though we 
> didn't come to
>  >  >  > >  >  > >  > agreement on how configuration parameters should look 
> like.
>  >  >  > >  >  > >
>  >  >  > >  >  > >  Why? Was there a discussion on the list about it? The
>  >  >  > >  >  > >  configuration is a bit unusual. Other stonith agents take 
> named
>  >  >  > >  >  > >  parameters. Though this kind of configuration also works, 
> I'd
>  >  >  > >  >  > >  prefer something similar to the others, e.g.
>  >  >  > >  >  >
>  >  >  > >  >  > Yes there was a discussion
>  >  >  > >  >  > 
> http://lists.community.tummy.com/pipermail/linux-ha-dev/2007-February/
>  >  >  > >  >
>  >  >  > >  >  It's a long one and peters out inconclusively.
>  >  >  > >  >
>  >  >  > >  >
>  >  >  > >  >  > See "new stonith external plugin". The config parameter 
> hostslist is
>  >  >  > >  >  > actually derived from original ssh plugin. I needed to have 
> a full
>  >  >  > >  >  > list of all controlled nodes and preferred to have it as one
>  >  >  > >  >  > parameter.
>  >  >  > >  >  >
>  >  >  > >  >  > >
>  >  >  > >  >  > >  hostname dom0 (or xenhost) config
>  >  >  > >  >  > >
>  >  >  > >  >  >
>  >  >  > >  >  > That would work if I needed just a dom0 host, but I also 
> need a list
>  >  >  > >  >  > of controlled nodes and probably configuration files.
>  >  >  > >  >
>  >  >  > >  >  That's why you can have several instances of a stonith resource
>  >  >  > >  >  (see e.g. external/ipmi). Each of them would run with different
>  >  >  > >  >  parameters. What I meant was:
>  >  >  > >  >
>  >  >  > >  >  hostname: xen vm
>  >  >  > >  >  dom0: xen dom0
>  >  >  > >  >  config: vm configuration file
>  >  >  > >  >
>  >  >  > >
>  >  >  > >  That's possible and easy to do but I'm not sure that it'll be 
> better.
>  >  >  > >  Current version allows to configure a clone. New version would 
> require
>  >  >  > >  configuring a separate resource for each node and creating 
> location
>  >  >  > >  constraints for each of them. Per my opinion that would be more
>  >  >  > >  complex configuration. May be I'm missing something.
>  >  >  > >
>  >  >  >
>  >  >  >
>  >  >  >
>  >  >  > --
>  >  >  > Serge Dubrouski.
>  >  >
>  >  >
>  >  >
>  >  > > _______________________________________________
>  >  >  > Linux-HA mailing list
>  >  >  > [email protected]
>  >  >  > http://lists.linux-ha.org/mailman/listinfo/linux-ha
>  >  >  > See also: http://linux-ha.org/ReportingProblems
>  >  >
>  >  >  --
>  >  >  Dejan
>  >  >
>  >  >
>  >  > _______________________________________________
>  >  >  Linux-HA mailing list
>  >  >  [email protected]
>  >  >  http://lists.linux-ha.org/mailman/listinfo/linux-ha
>  >  >  See also: http://linux-ha.org/ReportingProblems
>  >  >
>  >
>  >
>  >
>  >
>  > --
>  >
>  > Serge Dubrouski.
>  >
>  > _______________________________________________
>  >  Linux-HA mailing list
>  >  [email protected]
>  >  http://lists.linux-ha.org/mailman/listinfo/linux-ha
>  >  See also: http://linux-ha.org/ReportingProblems
>  >
>  >
>
> _______________________________________________
>  Linux-HA mailing list
>  [email protected]
>  http://lists.linux-ha.org/mailman/listinfo/linux-ha
>  See also: http://linux-ha.org/ReportingProblems
>



-- 
Serge Dubrouski.
_______________________________________________
Linux-HA mailing list
[email protected]
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems

Reply via email to