[Pacemaker] resource stop timeout broken in 1.0 branch tip

[Andrew Beekhof]

On Fri, Oct 8, 2010 at 4:17 PM, Lars Ellenberg
<lars.ellenberg at linbit.com> wrote:
> On Wed, Oct 06, 2010 at 06:29:12PM +0900, Keisuke MORI wrote:
>> 2010/10/6 Andrew Beekhof <andrew at beekhof.net>:
>> >> Is there more changesets
>> >> that need to be backported regarding to this issues?
>> >
>> > There is now that Andreas brought the problem to my attention :-)
>> >   http://hg.clusterlabs.org/pacemaker/1.1/rev/e097c70226fe
>> >
>> >> If not, I think that the Andreas' patch should be applied to 1.0.
>> >> It seems to me that the patch is sane as it would restore the old
>> >> behavior for the stop operation with having the resource attributes as
>> >> the first patch intended.
>> >
>> > See the comment in the above patch. Andreas' original patch wouldn't
>> > have worked if the resource definition changed.
>>
>> I see, I will backport this to 1.0 too.
>
> May I take the oportunity to point you to
> http://hg.clusterlabs.org/pacemaker/1.1/rev/3f8df3dfb328

ACK, no objection to this being backported :-)

>
> The important part being
>  (*hb_conn)->llc_ops->set_sendq_len(*hb_conn, 1024))
>
> It really improves heartbeat interaction,
> default qlen being 64.
>
> --
> : Lars Ellenberg
> : LINBIT | Your Way to High Availability
> : DRBD/HA support and consulting http://www.linbit.com
>
> DRBD® and LINBIT® are registered trademarks of LINBIT, Austria.
>
> _______________________________________________
> 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://developerbugs.linux-foundation.org/enter_bug.cgi?product=Pacemaker
>