On Thu, Feb 10, 2011 at 12:28 PM, Dejan Muhamedagic <deja...@fastmail.fm> wrote:
> On Thu, Feb 10, 2011 at 09:29:41AM +0100, Andrew Beekhof wrote:
>> On Thu, Feb 10, 2011 at 9:14 AM, Ulrich Windl
>> <ulrich.wi...@rz.uni-regensburg.de> wrote:
>> > Hi!
>> >
>> > I'm getting into Linux-HA, and it seems the "documentation" was made with 
>> > a very "hot needle". For example, ocf:pacemaker:ping has the following 
>> > documentation ("crm(live)ra# info ocf:ping" in SLES11 SP3+Updates):
>> >> crm(live)ra# info ocf:ping
>> >>     PID file
>> >>
>> >> dampen (integer, [5s]): Dampening interval
>> >>     The time to wait (dampening) further changes occur
>> >
>> > The sentence above is grammatically wrong.
>>
>> At least its spelt correctly, which is more than people usually get from me.
>>
>> >
>> >>
>> >> name (string, [pingd]): Attribute name
>> >>     The name of the attributes to set.  This is the name to be used in 
>> >> the constraints.
>> >>
>> >> multiplier (integer): Value multiplier
>> >>     The number by which to multiply the number of connected ping nodes by
>> >
>> > Please explain the reason (semantics) for the multiplication!
>>
>> Please read "Pacemaker Explained"
>>
>> >
>> >>
>> >> host_list* (string): Host list
>> >>     The list of ping nodes to count.
>> >
>> > "to count", or "to ping", or both?
>>
>> My, we are pedantic today.
>>
>> "To ping and count towards the number of active hosts"
>>
>> >> attempts (integer, [2]): no. of ping attempts
>> >>     Number of ping attempts, per host, before declaring it dead
>> >>
>> >> timeout (integer, [2]): ping timeout in seconds
>> >>     How long, in seconds, to wait before declaring a ping lost
>> >
>> > "a ping lost" == "a host dead"?
>>
>> Yes
>>
>> > Or is it the "the monitor reports a failure"?
>>
>> No
>>
>> >> options (string): Extra Options
>> >>     A catch all for any other options that need to be passed to ping.
>> >
>> > What about "Additional options to be passed to ping"?
>>
>> This text is too short, this text is too long... are you ever happy?
>>
>> >
>> >>
>> >> debug (string, [false]): Verbose logging
>> >>     Enables to use default attrd_updater verbose logging on every call.
>> >
>> > What about "`true' enables verbose logging"?
>> >
>> >>
>> >> Operations' defaults (advisory minimum):
>> >>
>> >>     start         timeout=60
>> >>     stop          timeout=20
>> >>     reload        timeout=100
>> >>     monitor_0     interval=10 timeout=60
>> >
>> > Is it really "monitor_0"? What is that "_0"?
>>
>> I'm guessing a typo
>
> Not a typo, but monitor with depth check 0. Well, perhaps
> appending depth could be skipped in this case.

Oh, I mistook it for a non-recurring monitor op
_______________________________________________
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems

Reply via email to