[Pacemaker] Last chance to object to the syntax for cluster tickets (multi-site clusters)

Gao,Yan ygao at suse.com
Fri Feb 24 04:38:12 EST 2012


Hi Florian,

On 02/24/12 16:00, Florian Haas wrote:
> On 02/24/12 02:53, Andrew Beekhof wrote:
>> We're about to lock in the syntax for cluster tickets (used for
>> multi-ste clusters).
>>
>> The syntax rules are at:
>>   https://github.com/gao-yan/pacemaker/commit/9e492f6231df2d8dd548f111a2490f02822b29ea
>>
>> And its use, along with some examples, can be found here:
>>    https://github.com/gao-yan/pacemaker/commit/5f75da8d99171cc100e87935c8c3fd2f83243f93
>>
>> If there are any comments/concerns, now is the time to raise them.
> 
> For naming, I must confess I find it a bit strange that while all other
> constraint types use ordinary English names ("order", "location",
> "colocation"), this one uses a rather strange looking abbreviation.
> However, I'll also concede that the only alternative that currently
> comes to my mind would be to rename the constraint type to "ticket", but
> that obviously creates ambiguity between ticket the constraint and
> ticket the thing that booth manages, so it would probably be worse.
I used to adopt this on developing. Indeed, that would be confusing.

> Perhaps others have a better idea.
Another one I used to adopt is "ticket_dep" which is somewhat awkward
too. Finally "rsc_ticket" was chosen because it has "rsc_" prefix which
looks consistent.

> 
> About the documentation, I generally find it very useful; I only have
> one addition for a suggestion: it's not immediately clear from the
> existing docs that multiple resources can depend on the same ticket. It
> does mention resource sets (which, still, could use an additional
> sentence à la "thus, multiple resources can depend on the same ticket"
> as a courtesy to the novice reader), but it doesn't say whether it's OK
> to have multiple constraints referring to the same ticket.
Thanks for pointing it out! Added the description:
https://github.com/gao-yan/pacemaker/commit/25dc030872a101b0592646d1a57860494753760a

> 
> If I can spare the time some time in the next few weeks I might also
> prepare a "die, passive voice, die" patch for that documentation page,
> but that's just a pet peeve of mine. :)
Any revise of the docs is welcome, as long as that improves the reading
experience. :)

Regards,
  Gao,Yan
-- 
Gao,Yan <ygao at suse.com>
Software Engineer
China Server Team, SUSE.




More information about the Pacemaker mailing list