[ClusterLabs Developers] Validation of resource meta-attributes

[Klaus Wenninger]

On 09/18/2017 01:26 PM, Oyvind Albrigtsen wrote:
> On 18/09/17 11:09 +0000, Ondrej Mular wrote:
>> Hi all,
>>
>> during summit in Nuremberg, there was a discussion about improving
>> usability. One of mentioned areas was validation of resource meta-
>> attributes.
>>
>> There was an idea to have meta-attributes as element attributes of
>> resource elements (primitive/group/clone/master/bundle) in CIB. With
>> this approach meta-attributes will be validated directly against CIB
>> scheme. But this idea was rejected right away because it doesn't work
>> with meta-attributes used with rules.
>>
>> Maybe better way to solve this would be to do it in similar fashion as
>> cluster properties. Pacemaker provides definition of cluster properties
>> in same XML metadata format as resource-agents have.

Instance attributes handled by stonithd are another place where a
similar approach is being used already.
(`/usr/libexec/pacemaker/stonithd metadata`)

Regards,
Klaus
 
>>
>> If we can obtain definition of meta-attributes metadata from pacemaker
>> in similar format, then each CLI can validate meta-attributes against
>> this definition and in GUI setting of them can be more user friendly.
>> Also, we can take advantage of this format and put there some
>> understandable description (maybe also name/label of meta-attribute)
>> and much more. Actually, marx is working on some improvements of ocf
>> XML metadata format so we can have more useful information for UIs.
>>
>> Currently, pcs doesn't do any validation of resource meta-attributes.
>> We were thinking about some validation, but we will have to hardcode it
>> into pcs and update it every time some changes happen in pacemaker.
>> With metadata approach, change in definition would be sufficient.
>>
>> What is your opinion on this?
> I think validation based on metadata definition would be the best way
> to do it.
>>
>> Ondrej
>>
>> _______________________________________________
>> Developers mailing list
>> Developers at clusterlabs.org
>> http://lists.clusterlabs.org/mailman/listinfo/developers
>
> _______________________________________________
> Developers mailing list
> Developers at clusterlabs.org
> http://lists.clusterlabs.org/mailman/listinfo/developers