[Pacemaker] again "return code", now in crm_attribute

[Andrew Beekhof]

On 10 Jan 2014, at 3:51 pm, Andrey Groshev <greenx at yandex.ru> wrote:

> 
> 
> 10.01.2014, 03:28, "Andrew Beekhof" <andrew at beekhof.net>:
>> On 9 Jan 2014, at 4:44 pm, Andrey Groshev <greenx at yandex.ru> wrote:
>> 
>>>  09.01.2014, 02:39, "Andrew Beekhof" <andrew at beekhof.net>:
>>>>   On 18 Dec 2013, at 11:55 pm, Andrey Groshev <greenx at yandex.ru> wrote:
>>>>>    Hi, Andrew and ALL.
>>>>> 
>>>>>    I'm sorry, but I again found an error. :)
>>>>>    Crux of the problem:
>>>>> 
>>>>>    # crm_attribute --type crm_config --attr-name stonith-enabled --query; echo $?
>>>>>    scope=crm_config  name=stonith-enabled value=true
>>>>>    0
>>>>> 
>>>>>    # crm_attribute --type crm_config --attr-name stonith-enabled --update firstval ; echo $?
>>>>>    0
>>>>> 
>>>>>    # crm_attribute --type crm_config --attr-name stonith-enabled --query; echo $?
>>>>>    scope=crm_config  name=stonith-enabled value=firstval
>>>>>    0
>>>>> 
>>>>>    # crm_attribute --type crm_config --attr-name stonith-enabled  --update secondval --lifetime=reboot ; echo $?
>>>>>    0
>>>>> 
>>>>>    # crm_attribute --type crm_config --attr-name stonith-enabled --query; echo $?
>>>>>    scope=crm_config  name=stonith-enabled value=firstval
>>>>>    0
>>>>> 
>>>>>    # crm_attribute --type crm_config --attr-name stonith-enabled  --update thirdval --lifetime=forever ; echo $?
>>>>>    0
>>>>> 
>>>>>    # crm_attribute --type crm_config --attr-name stonith-enabled --query; echo $?
>>>>>    scope=crm_config  name=stonith-enabled value=firstval
>>>>>    0
>>>>> 
>>>>>    Ie if specify the lifetime of an attribute, then a attribure is not updated.
>>>>> 
>>>>>    If impossible setup the lifetime of the attribute when it is installing, it must be return an error.
>>>>   Agreed. I'll reproduce and get back to you.
>>>  How, I was able to review code, problem comes when used both options "--type" and options "--lifetime".
>>>  One variant in "case" without break;
>>>  Unfortunately, I did not have time to dive into the logic.
>> 
>> Actually, the logic is correct.  The command:
>> 
>> # crm_attribute --type crm_config --attr-name stonith-enabled  --update secondval --lifetime=reboot ; echo $?
>> 
>> is invalid.  You only get to specify --type OR --lifetime, not both.
>> By specifying --lifetime, you're creating a node attribute, not a cluster proprerty.
> 
> With this, I do not argue. I think that should be the exit code is NOT ZERO, ie it's error!

No, its setting a value, just not where you thought (or where you're looking for it in the next command).

Its the same as writing:

  crm_attribute --type crm_config --type status --attr-name stonith-enabled  --update secondval; echo $?

Only the last value for --type wins

> 
>>>>>    And if possible then the value should be established.
>>>>>    In general, something is wrong.
>>>>>    Denser unfortunately not yet looked, because I struggle with "STONITH" :)
>>>>> 
>>>>>    P.S. Andrew! Late to congratulate you on your new addition to the family.
>>>>>    This fine time - now you will have toys which was not in your childhood.
>>>>> 
>>>>>    _______________________________________________
>>>>>    Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
>>>>>    http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>>>>> 
>>>>>    Project Home: http://www.clusterlabs.org
>>>>>    Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
>>>>>    Bugs: http://bugs.clusterlabs.org
>>>>   ,
>>>>   _______________________________________________
>>>>   Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
>>>>   http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>>>> 
>>>>   Project Home: http://www.clusterlabs.org
>>>>   Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
>>>>   Bugs: http://bugs.clusterlabs.org
>>>  _______________________________________________
>>>  Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
>>>  http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>>> 
>>>  Project Home: http://www.clusterlabs.org
>>>  Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
>>>  Bugs: http://bugs.clusterlabs.org
>> 
>> ,
>> _______________________________________________
>> Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
>> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>> 
>> Project Home: http://www.clusterlabs.org
>> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
>> Bugs: http://bugs.clusterlabs.org
> 
> _______________________________________________
> Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
> 
> Project Home: http://www.clusterlabs.org
> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
> Bugs: http://bugs.clusterlabs.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 841 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <https://lists.clusterlabs.org/pipermail/pacemaker/attachments/20140110/4c5e8cce/attachment-0003.sig>