Hi Dejan,
Hi All,

We reconsidered a patch.....



In Pacemaker1.1, node names in STONITH are always small letters.
When a user uses a capital letter in host name, STONITH of libvirt fails.

This patch lets STONITH by libvirt succeed in the next setting.

 * host name(upper case) and hostlist(upper case) and domain_id on 
libvirt(uppper case)
 * host name(upper case) and hostlist(lower case) and domain_id on 
libvirt(lower case)
 * host name(lower case) and hostlist(upper case) and domain_id on 
libvirt(uppper case)
 * host name(lower case) and hostlist(lower case) and domain_id on 
libvirt(lower case)


However, in the case of the next setting, STONITH of libvirt causes an error.
In this case it is necessary for the user to make the size of the letter of the 
host name to manage of libvirt the same as host name to appoint in hostlist.

 * host name(upper case) and hostlist(lower case) and domain_id on 
libvirt(uppper case)
 * host name(upper case) and hostlist(uppper case) and domain_id on 
libvirt(lower case)
 * host name(lower case) and hostlist(lower case) and domain_id on 
libvirt(uppper case)
 * host name(lower case) and hostlist(uppper case) and domain_id on 
libvirt(lower case)


This patch is effective for letting STONITH by libvirt when host name was set 
for a capital letter succeed.

Best Regards,
Hideo Yamauchi.




----- Original Message -----
> From: "[email protected]" <[email protected]>
> To: Cluster Labs - All topics related to open-source clustering welcomed 
> <[email protected]>
> Cc: 
> Date: 2015/9/15, Tue 03:28
> Subject: Re: [ClusterLabs] [Patch][glue][external/libvirt] Conversion to a 
> lower case of hostlist.
> 
> Hi Dejan,
> 
>>  I suppose that you'll send another one? I can vaguelly recall
>>  a problem with non-lower case node names, but not the specifics.
>>  Is that supposed to be handled within a stonith agent?
> 
> 
> Yes.
> We make a different patch now.
> With the patch, I solve a node name of the non-small letter in the range of 
> stonith agent.
> # But the patch cannot cover all all patterns.
> 
> Please wait a little longer.
> I send a patch again.
> For a new patch, please tell me your opinion.
> 
> Best Regards,
> Hideo Yamauchi.
> 
> 
> 
> ----- Original Message -----
>>  From: Dejan Muhamedagic <[email protected]>
>>  To: ClusterLabs-ML <[email protected]>
>>  Cc: 
>>  Date: 2015/9/14, Mon 22:20
>>  Subject: Re: [ClusterLabs] [Patch][glue][external/libvirt] Conversion to a 
> lower case of hostlist.
>> 
>>  Hi Hideo-san,
>> 
>>  On Tue, Sep 08, 2015 at 05:28:05PM +0900, [email protected] wrote:
>>>   Hi All,
>>> 
>>>   We intend to change some patches.
>>>   We withdraw this patch.
>> 
>>  I suppose that you'll send another one? I can vaguelly recall
>>  a problem with non-lower case node names, but not the specifics.
>>  Is that supposed to be handled within a stonith agent?
>> 
>>  Cheers,
>> 
>>  Dejan
>> 
>>>   Best Regards,
>>>   Hideo Yamauchi.
>>> 
>>> 
>>>   ----- Original Message -----
>>>   > From: "[email protected]" 
>>  <[email protected]>
>>>   > To: ClusterLabs-ML <[email protected]>
>>>   > Cc: 
>>>   > Date: 2015/9/7, Mon 09:06
>>>   > Subject: [ClusterLabs] [Patch][glue][external/libvirt] Conversion 
> to a 
>>  lower case of hostlist.
>>>   > 
>>>   > Hi All,
>>>   > 
>>>   > When a cluster carries out stonith, Pacemaker handles host name 
> by a 
>>  small 
>>>   > letter.
>>>   > When a user sets the host name of the OS and host name of 
> hostlist of 
>>>   > external/libvrit in capital letters and waits, stonith is not 
> carried 
>>  out.
>>>   > 
>>>   > The external/libvrit to convert host name of hostlist, and to 
> compare 
>>  can assist 
>>>   > a setting error of the user.
>>>   > 
>>>   > Best Regards,
>>>   > Hideo Yamauchi.
>>>   > 
>>>   > _______________________________________________
>>>   > Users mailing list: [email protected]
>>>   > http://clusterlabs.org/mailman/listinfo/users
>>>   > 
>>>   > Project Home: http://www.clusterlabs.org
>>>   > Getting started: 
>>  http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
>>>   > Bugs: http://bugs.clusterlabs.org
>>>   > 
>>> 
>>>   _______________________________________________
>>>   Users mailing list: [email protected]
>>>   http://clusterlabs.org/mailman/listinfo/users
>>> 
>>>   Project Home: http://www.clusterlabs.org
>>>   Getting started: 
> http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
>>>   Bugs: http://bugs.clusterlabs.org
>> 
>>  _______________________________________________
>>  Users mailing list: [email protected]
>>  http://clusterlabs.org/mailman/listinfo/users
>> 
>>  Project Home: http://www.clusterlabs.org
>>  Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
>>  Bugs: http://bugs.clusterlabs.org
>> 
> 
> _______________________________________________
> Users mailing list: [email protected]
> http://clusterlabs.org/mailman/listinfo/users
> 
> Project Home: http://www.clusterlabs.org
> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
> Bugs: http://bugs.clusterlabs.org
> 

Attachment: trac3122.patch
Description: Binary data

_______________________________________________
Users mailing list: [email protected]
http://clusterlabs.org/mailman/listinfo/users

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org

Reply via email to